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
[Charts] Use the Edit Properties modal throughout React views #9267
Conversation
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.
Couple o' random questions, but LGTM.
Codecov Report
@@ Coverage Diff @@
## master #9267 +/- ##
==========================================
- Coverage 58.9% 58.88% -0.02%
==========================================
Files 373 373
Lines 12026 12069 +43
Branches 2953 2957 +4
==========================================
+ Hits 7084 7107 +23
- Misses 4763 4783 +20
Partials 179 179
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.
LGTM
CATEGORY
Choose one
SUMMARY
The Edit Properties modal previously belonging only to the Explore dropdown is now used in the two additional places. This change replaces links to the CRUD edit page for charts.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
TEST PLAN
ADDITIONAL INFORMATION
REVIEWERS
@nytai @etr2460 @rusackas