-
Notifications
You must be signed in to change notification settings - Fork 39
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
Switch seeddb filter forms to new way of uncrispifying #3126
Switch seeddb filter forms to new way of uncrispifying #3126
Conversation
🦙 MegaLinter status: ✅ SUCCESS
See detailed report in MegaLinter reports |
6a733a3
to
b4db6dc
Compare
Test results 8 files 8 suites 8m 14s ⏱️ Results for commit d037671. ♻️ This comment has been updated with latest results. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #3126 +/- ##
==========================================
- Coverage 60.44% 60.44% -0.01%
==========================================
Files 605 605
Lines 43757 43749 -8
Branches 48 48
==========================================
- Hits 26451 26443 -8
Misses 17294 17294
Partials 12 12 ☔ View full report in Codecov by Sentry. |
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.
not 100% sure I understand everything, but it seems reasonable and Room filtering and Type filtering seems to work same as before so 👍
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.
Nice! Some suggestions for simplification sake below
ef20131
to
c383bb7
Compare
This was removed when uncrispifying seedDB This adds a new uncrispified version of this method
c383bb7
to
d037671
Compare
Quality Gate passedIssues Measures |
Closes #2995.
Reverts things done in #2981 and #3092 (
ManagementProfileForm
will also be changed to useget_single_layout
again).