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(dashboard): View query of the chart in dashboard #14667
feat(dashboard): View query of the chart in dashboard #14667
Conversation
thanks for the improvement!! this is great! @m-ajay does the query get updated when filters are applied? |
/testenv up |
@junlincc Yes, it respects the filters applied in dashboard. |
Codecov Report
@@ Coverage Diff @@
## master #14667 +/- ##
=======================================
Coverage 77.54% 77.54%
=======================================
Files 958 959 +1
Lines 48541 48552 +11
Branches 5702 5708 +6
=======================================
+ Hits 37640 37650 +10
- Misses 10700 10702 +2
+ Partials 201 200 -1
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
@junlincc Ephemeral environment spinning up at http://54.245.153.87:8080. Credentials are |
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
Ephemeral environment shutdown and build artifacts deleted. |
@junlincc I don't think it's related to this issue. Seems like a separate issue. |
* Extract view query component out * Typo correction * Fix and rename tests * Remove logs, and test code * Rename file - fix tests * Fix a typo * Linting errors, add Apache License Header
* Extract view query component out * Typo correction * Fix and rename tests * Remove logs, and test code * Rename file - fix tests * Fix a typo * Linting errors, add Apache License Header
* Extract view query component out * Typo correction * Fix and rename tests * Remove logs, and test code * Rename file - fix tests * Fix a typo * Linting errors, add Apache License Header
SUMMARY
Lets the user view query used to generate the chart from the dashboard.
BEFORE/AFTER SCREENSHOTS
View Query in the menu option
View the query
TEST PLAN
Manual and CI
ADDITIONAL INFORMATION