Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Fix filter set via URL isn't persistent if set on the list page #6504

Merged
merged 2 commits into from
Aug 12, 2021
Merged

Fix filter set via URL isn't persistent if set on the list page #6504

merged 2 commits into from
Aug 12, 2021

Conversation

fzaninotto
Copy link
Member

Closes #6502

@fzaninotto fzaninotto added the RFR Ready For Review label Aug 11, 2021
@diegocasmo
Copy link

It looks like we chose to tackle this issue almost at the same time 🙂 (#6505). I'll go ahead and close my PR in favor of yours, as this one also removes an unnecessary dispatch which I didn't caught.

@fzaninotto fzaninotto merged commit b0b99a9 into master Aug 12, 2021
@fzaninotto fzaninotto deleted the fix-uselistparams-url branch August 12, 2021 07:05
@fzaninotto fzaninotto added this to the 3.17.2 milestone Aug 12, 2021
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
RFR Ready For Review
Projects
None yet
Milestone
3.17.2
Development

Successfully merging this pull request may close these issues.

When changed via the URLs, filters aren't persistent
2 participants
@fzaninotto @diegocasmo