innovark37 opened a new pull request, #40885: URL: https://github.com/apache/superset/pull/40885
### SUMMARY Adds XLSX as an output format for chart-based email reports. This change: - Adds XLSX options to the chart report and Alerts & Reports modals. - Generates XLSX files using post-processed chart data. - Preserves the configured Excel export and index behavior. - Attaches regular exports as `.xlsx` files. - Attaches server-paginated, multi-file exports as `.zip` archives. - Sets the appropriate MIME types for XLSX and ZIP attachments. - Adds backend and frontend unit test coverage. - Updates the Alerts & Reports documentation. Dashboard reports remain limited to PDF and PNG formats. ### BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF **Before** XLSX was not available as an email report format. <!-- Add a screenshot of the format selector without XLSX. --> **After** Chart reports can be delivered as XLSX email attachments. <!-- Add screenshots of XLSX in: 1. The chart email report modal. 2. The Alerts & Reports modal. --> ### TESTING INSTRUCTIONS 1. Enable the `ALERT_REPORTS` feature flag and configure email reports. 2. Open a chart and select **Schedule email report**. 3. Select **Excel (XLSX) attached in email** as the report format. 4. Create the report and trigger its execution. 5. Verify that the email contains an `.xlsx` attachment that opens successfully. 6. Verify that the exported data reflects the chart's post-processing. 7. Repeat with a server-paginated chart and verify that the email contains a `.zip` archive with XLSX files. 8. Open **Settings > Alerts & Reports** and verify that XLSX is available for chart reports but not dashboard reports. 9. Verify that the screenshot width field is not displayed when XLSX is selected. ### ADDITIONAL INFORMATION <!--- Check any relevant boxes with "x" --> <!--- HINT: Include "Fixes #nnn" if you are fixing an existing issue --> - [ ] Has associated issue: - [x] Required feature flags: `ALERT_REPORTS` - [x] Changes UI - [ ] Includes DB Migration (follow approval process in [SIP-59](https://github.com/apache/superset/issues/13351)) - [ ] Migration is atomic, supports rollback & is backwards-compatible - [ ] Confirm DB migration upgrade and downgrade tested - [ ] Runtime estimates and downtime expectations provided - [x] Introduces new feature or API - [ ] Removes existing feature or API -- 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]
