Skip to content
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

ui: Preserve search/sorting on Statements page #46575

Closed

Conversation

dhartunian
Copy link
Collaborator

Resolves: #45404

Search query and sorting params are preserved as URL search
params. It doesn't break existing routing configuration and
allows restore custom state from URL.

Release justification: bug fixes and low-risk updates to new functionality

koorosh added 3 commits March 19, 2020 14:27
Search query and sorting params are preserved as URL search
params. It doesn't break existing routing configuration and
allows restore custom state from URL.

Release note: None

Release justification: bug fixes and low-risk updates to new functionality
Before, URL search params were removed when App filter changed,
and this behavior introduced inconsistent state between defined
URL params and actual state.

Now, when App filter changes, URL search params updated as well.

Release note: None

Release justification: bug fixes and low-risk updates to new functionality
Before, on Statements page when user clears search statement,
`q` url search param is not cleared

Now, URL search param is cleared as well and preserves correct
state.

Release note: None

Release justification: bug fixes and low-risk updates to new functionality
@dhartunian dhartunian added the do-not-merge bors won't merge a PR with this label. label Mar 25, 2020
@cockroach-teamcity
Copy link
Member

This change is Reviewable

@dhartunian dhartunian closed this Apr 21, 2020
@dhartunian dhartunian deleted the contractor-interview-pr-1 branch April 21, 2020 14:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
do-not-merge bors won't merge a PR with this label.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

ui: persist searches and sorts on statement list page
3 participants