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: