-
Notifications
You must be signed in to change notification settings - Fork 353
feat: Add saved query support to dashboards #1584
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
Fixes: HDX-1717
|
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
PR Review: Add saved query support to dashboards✅ Overall Implementation: The feature is well-implemented with comprehensive E2E tests. Issues Found:
Recommendations:
Status: Approve with minor suggestions for follow-up. |
E2E Test Results❌ 2 tests failed • 61 passed • 4 skipped • 766s
Tests ran across 4 shards in parallel. |
|
Hey @teeohhem, I’d like to suggest moving the Save query and filters action into the global Dashboard Right now we have a mixed interaction model: the dashboard layout (tiles) autosaves, but query and filter inputs require a manual save via the floppy disk icon. This can be confusing. Two main UX issues with the current placement:
Moving this into the Dashboard
The menu could be state-based. State 1: No defaults saved:[ Menu]
----------------------------------
⬇️ Export Dashboard
⬆️ Import New Dashboard
----------------------------------
💾 Save Query & Filters as Default
----------------------------------
🗑️ Delete Dashboard (danger text)State 2: Defaults are active[ Menu ]
----------------------------------
⬇️ Export Dashboard
⬆️ Import New Dashboard
----------------------------------
💾 Update Default Query & Filters
✖️ Remove Default Query & Filters (danger text)
----------------------------------
🗑️ Delete Dashboard (danger text)This is only a suggestion. Let me know what you think. |
Fixes: HDX-1717
Adds: