-
Notifications
You must be signed in to change notification settings - Fork 13.6k
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
feat: View all fav charts and dashboards #12060
Conversation
Codecov Report
@@ Coverage Diff @@
## master #12060 +/- ##
==========================================
- Coverage 64.71% 63.25% -1.46%
==========================================
Files 960 478 -482
Lines 47302 29445 -17857
Branches 4629 0 -4629
==========================================
- Hits 30611 18626 -11985
+ Misses 16517 10819 -5698
+ Partials 174 0 -174
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This LGTM! I think the follow-up PR addresses the only remaining concern brought up on this.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Thanks making those changes
This finally saved my day, thanks guys. |
SUMMARY
Allows to show fav Charts and Dashboards from the Welcome page, by clicking the "Favorite" tab and "View all". It also removes the "Favorite" tab for "Saved Queries" as this functionality is not implemented yet.
TEST PLAN
ADDITIONAL INFORMATION