Skip to content

feat(ourlogs): add trackAnalytics calls for new export modal#113611

Merged
JoshuaKGoldberg merged 53 commits intomasterfrom
ourlogs-export-modal-telemetry
May 1, 2026
Merged

feat(ourlogs): add trackAnalytics calls for new export modal#113611
JoshuaKGoldberg merged 53 commits intomasterfrom
ourlogs-export-modal-telemetry

Conversation

@JoshuaKGoldberg
Copy link
Copy Markdown
Member

Updates the existing string literals to not hardcode in "csv" as the export type.

Fixes LOGS-701.

JoshuaKGoldberg and others added 30 commits April 10, 2026 11:01
Add DataExportWithModal (global modal + scraps form) beside the existing logs export control. Extend useDataExport to send an optional top-level limit to the data-export API.

Co-Authored-By: Cursor <noreply@cursor.com>
Made-with: Cursor
@JoshuaKGoldberg JoshuaKGoldberg removed request for a team April 24, 2026 17:35
Comment thread static/app/views/explore/logs/exports/logsExportModalButton.tsx Outdated
Copy link
Copy Markdown
Contributor

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

There are 2 total unresolved issues (including 1 from previous review).

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 8117c74. Configure here.

Comment thread static/app/views/explore/logs/exports/logsExportModalButton.tsx
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Do Not Merge Don't merge Scope: Backend Automatically applied to PRs that change backend components Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants