-
-
Notifications
You must be signed in to change notification settings - Fork 7.6k
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 sortable config disabled at launch #9115
Fix sortable config disabled at launch #9115
Conversation
Codecov Report
@@ Coverage Diff @@
## master #9115 +/- ##
=======================================
Coverage 34.64% 34.64%
=======================================
Files 1308 1308
Lines 14431 14431
Branches 1432 1432
=======================================
+ Hits 4999 5000 +1
+ Misses 8518 8517 -1
Partials 914 914
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
Fix works but it does dump the following warning to the console:
|
When do you get this message ? |
Waiting for the explanation on the deep filtering message before merging @derrickmehaffy @petersg83 |
Yep :) It's when you sort by a RF just. It's not specific to this PR |
Sortable param was set to false for relational fields when restarting the app
fixes #9099