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: Remove default values for engine and schemas #15635
Conversation
@hughhhh did you test adding/editing a DB with this change? |
Codecov Report
@@ Coverage Diff @@
## master #15635 +/- ##
==========================================
+ Coverage 76.86% 76.98% +0.11%
==========================================
Files 977 979 +2
Lines 51343 51835 +492
Branches 6920 7064 +144
==========================================
+ Hits 39466 39906 +440
- Misses 11654 11702 +48
- Partials 223 227 +4
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.
+1
🏷 2021.27 |
* remove default values * don't set initial value on save * set defaults for engine_params * update with JSON.parse (cherry picked from commit 2252f33)
* remove default values * don't set initial value on save * set defaults for engine_params * update with JSON.parse
* remove default values * don't set initial value on save * set defaults for engine_params * update with JSON.parse
* remove default values * don't set initial value on save * set defaults for engine_params * update with JSON.parse
SUMMARY
Remove putting a default JSON object whenever engine_params or metadata_params is set. Also removed the allow_schemas_csv_upload quotation marks on that field
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
TESTING INSTRUCTIONS
ADDITIONAL INFORMATION