-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Show applied filter #6225
Show applied filter #6225
Conversation
@kirya-dev Test are failing because of the deprecation. You have to add a |
Hmm. Test required for check default logic.. Test has others assert not related with isDefaultFilter |
Co-authored-by: Vincent Langlet <[email protected]>
Co-authored-by: Vincent Langlet <[email protected]>
The changelog is not well formed. It will be nice to have some screenshots about before and after. Ad it would be nice if you explain what is the alternative to setting default filters now. Bonus point if you add it to the deprecation message |
Configure default filter was not change |
Thanks ! |
Subject
Change the UX-behavior to stop hidding default filter and considering them as normal filters.
Closes #6217.
Closes #6137.
Closes #5745.
Changelog