-
Notifications
You must be signed in to change notification settings - Fork 13.2k
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
fix: move metric parsing to state instantiation #15069
fix: move metric parsing to state instantiation #15069
Conversation
/testenv up |
@eschutho Ephemeral environment creation is currently limited to committers. |
/testenv up |
@hughhhh Ephemeral environment spinning up at http://54.212.72.236:8080. Credentials are |
Codecov Report
@@ Coverage Diff @@
## master #15069 +/- ##
=======================================
Coverage 77.78% 77.78%
=======================================
Files 966 966
Lines 49629 49629
Branches 6314 6314
=======================================
Hits 38604 38604
Misses 10824 10824
Partials 201 201
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
Ephemeral environment shutdown and build artifacts deleted. |
🏷 2021.23 |
(cherry picked from commit 0c470fe)
SUMMARY
I noticed while testing that the markdown metric field isn't updating properly.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
After:
dataset.mov
TESTING INSTRUCTIONS
It's difficult to write any kind of tests for the ace editor when looking at value because it always renders "xxx". Something we can try to figure out in the future. In the meantime I added more tests around editing the other metrics.
Manual testing: click to edit a dataset, go to metrics, and try to add some test in the markdown field. It should show the text.
ADDITIONAL INFORMATION