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

UIIN-3099: Run history.replace once during component mount and update to avoid URL rewriting. #2662

Merged
merged 1 commit into from
Nov 1, 2024

Conversation

Dmytro-Melnyshyn
Copy link
Contributor

@Dmytro-Melnyshyn Dmytro-Melnyshyn commented Nov 1, 2024

Purpose

Send staffSuppress filter and sort value in URL at the same time to avoid rewriting during mount and update.

Approach

Set the necessary search params in const searchParams = new URLSearchParams(location.search); and only then do history.replace.

Refs

UIIN-3099

Screenshots

2024-11-01_14h58_47.mp4

Copy link

github-actions bot commented Nov 1, 2024

Jest Unit Test Statistics

       1 files  ±0     254 suites  ±0   15m 31s ⏱️ + 1m 3s
1 015 tests ±0  1 013 ✔️ ±0  2 💤 ±0  0 ±0 
1 022 runs  ±0  1 020 ✔️ ±0  2 💤 ±0  0 ±0 

Results for commit 85002d4. ± Comparison against base commit a7cbfe3.

Copy link

sonarcloud bot commented Nov 1, 2024

@Dmytro-Melnyshyn Dmytro-Melnyshyn merged commit 96b015c into master Nov 1, 2024
5 checks passed
@Dmytro-Melnyshyn Dmytro-Melnyshyn deleted the UIIN-3099 branch November 1, 2024 13:23
BogdanDenis pushed a commit that referenced this pull request Nov 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants