codeant-ai-for-open-source[bot] commented on PR #38513:
URL: https://github.com/apache/superset/pull/38513#issuecomment-4045996023

   ## **Sequence Diagram**
   
   This PR fixes legacy chart CSV and Excel downloads by generating a relative 
explore URL before posting form data, preventing double host URL construction. 
It also adds backend Excel file generation and unified export permission checks 
for both CSV and XLSX.
   
   ```mermaid
   sequenceDiagram
       participant User
       participant DashboardUI
       participant ExploreUtils
       participant Backend
       participant Security
       participant ExportGenerator
   
       User->>DashboardUI: Click Download CSV or Download Excel
       DashboardUI->>ExploreUtils: Call exportChart for legacy chart
       ExploreUtils->>ExploreUtils: Build relative explore URL with csv or xlsx 
flag
       ExploreUtils->>Backend: Post legacy export request
       Backend->>Security: Check data export permission
       Security-->>Backend: Permission granted
       Backend->>ExportGenerator: Generate csv or xlsx file from chart data
       ExportGenerator-->>DashboardUI: Return file download response
       DashboardUI-->>User: Start file download
   ```
   
   ---
   *Generated by [CodeAnt AI](https://codeant.ai)*
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to