Skip to content

docs(exploring-data): document CSV/XLSX download in drill modals - #42952

Merged
sadpandajoe merged 2 commits into
masterfrom
fix/from-pr-42051-needs-docs-feat-explore-add-csv-xls-down
Aug 10, 2026
Merged

docs(exploring-data): document CSV/XLSX download in drill modals#42952
sadpandajoe merged 2 commits into
masterfrom
fix/from-pr-42051-needs-docs-feat-explore-add-csv-xls-down

Conversation

@rusackas

Copy link
Copy Markdown
Member

SUMMARY

#42051 added a Download button (CSV/XLSX) to the Drill to Detail and Drill By modals, but this user-facing feature wasn't mentioned in the docs. This adds a short section to the exploring-data tutorial covering it.

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

Docs-only change, no UI screenshots.

TESTING INSTRUCTIONS

  • Read docs/docs/using-superset/exploring-data.mdx and confirm the new "Downloading Drill to Detail and Drill By Results" section renders correctly.
  • Optionally run the docs site locally (cd docs && npm start) and check the page.

ADDITIONAL INFORMATION

  • Has associated issue:
  • Required feature flags:
  • Changes UI
  • Includes DB Migration (follow approval process in SIP-59)
    • Migration is atomic, supports rollback & is backwards-compatible
    • Confirm DB migration upgrade and downgrade tested
    • Runtime estimates and downtime expectations provided
  • Introduces new feature or API
  • Removes existing feature or API

Related to #42051.

Co-Authored-By: Claude Sonnet 5 noreply@anthropic.com

The Drill to Detail and Drill By modals (#42051) added a Download
button for exporting the displayed rows as CSV or Excel, but this
wasn't reflected in the exploring-data tutorial docs.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@dosubot dosubot Bot added data:csv Related to import/export of CSVs doc:user User / Superset documentation labels Aug 10, 2026
@bito-code-review

bito-code-review Bot commented Aug 10, 2026

Copy link
Copy Markdown
Contributor

Code Review Agent Run #254fac

Actionable Suggestions - 0
Review Details
  • Files reviewed - 1 · Commit Range: e3f7d45..e3f7d45
    • docs/docs/using-superset/exploring-data.mdx
  • Files skipped - 0
  • Tools
    • Whispers (Secret Scanner) - ✔︎ Successful
    • Detect-secrets (Secret Scanner) - ✔︎ Successful

Bito Usage Guide

Commands

Type the following command in the pull request comment and save the comment.

  • /review - Manually triggers a full AI review.

  • /pause - Pauses automatic reviews on this pull request.

  • /resume - Resumes automatic reviews.

  • /resolve - Marks all Bito-posted review comments as resolved.

  • /abort - Cancels all in-progress reviews.

Refer to the documentation for additional commands.

Configuration

This repository uses Superset You can customize the agent settings here or contact your Bito workspace admin at evan@preset.io.

Documentation & Help

AI Code Review powered by Bito Logo

@github-actions github-actions Bot added doc Namespace | Anything related to documentation preset-io labels Aug 10, 2026
@netlify

netlify Bot commented Aug 10, 2026

Copy link
Copy Markdown

Deploy Preview for superset-docs-preview ready!

Name Link
🔨 Latest commit 82e383b
🔍 Latest deploy log https://app.netlify.com/projects/superset-docs-preview/deploys/6a795c97efa6960008f18796
😎 Deploy Preview https://deploy-preview-42952--superset-docs-preview.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

Comment thread docs/docs/using-superset/exploring-data.mdx Outdated
@bito-code-review

Copy link
Copy Markdown
Contributor

The flagged issue is correct. The current documentation implies that the export is limited to the visible page, whereas the implementation actually exports up to the global row limit. To resolve this, update the documentation to clarify that the export may contain a broader result set than the current page.

Suggested update for docs/docs/using-superset/exploring-data.mdx:

Use the **Download** button in the modal's toolbar to export the underlying row-level data as CSV or Excel (XLSX). Note that the export may contain a broader result set than the currently visible page.

There are no other comments on this PR to address.

docs/docs/using-superset/exploring-data.mdx

Use the **Download** button in the modal's toolbar to export the underlying row-level data as CSV or Excel (XLSX). Note that the export may contain a broader result set than the currently visible page.

…ble page

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@bito-code-review

bito-code-review Bot commented Aug 10, 2026

Copy link
Copy Markdown
Contributor

Code Review Agent Run #c96bb9

Actionable Suggestions - 0
Review Details
  • Files reviewed - 1 · Commit Range: e3f7d45..82e383b
    • docs/docs/using-superset/exploring-data.mdx
  • Files skipped - 0
  • Tools
    • Whispers (Secret Scanner) - ✔︎ Successful
    • Detect-secrets (Secret Scanner) - ✔︎ Successful

Bito Usage Guide

Commands

Type the following command in the pull request comment and save the comment.

  • /review - Manually triggers a full AI review.

  • /pause - Pauses automatic reviews on this pull request.

  • /resume - Resumes automatic reviews.

  • /resolve - Marks all Bito-posted review comments as resolved.

  • /abort - Cancels all in-progress reviews.

Refer to the documentation for additional commands.

Configuration

This repository uses Superset You can customize the agent settings here or contact your Bito workspace admin at evan@preset.io.

Documentation & Help

AI Code Review powered by Bito Logo

@sadpandajoe
sadpandajoe merged commit 5bcd9cd into master Aug 10, 2026
60 checks passed
@sadpandajoe
sadpandajoe deleted the fix/from-pr-42051-needs-docs-feat-explore-add-csv-xls-down branch August 10, 2026 17:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

data:csv Related to import/export of CSVs doc:user User / Superset documentation doc Namespace | Anything related to documentation preset-io size/XS

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants