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

Fix dashboard user page crash #5732

Merged
merged 2 commits into from
Jul 10, 2024

Conversation

thornbill
Copy link
Member

Changes

  • Fixes getLocationSearch when search params exist normally and in the hash of the url (i.e. https://example.com?foo=1#?bar=2)
  • Updates user pages to use the useSearchParams hook from react-router

Issues
Fixes #5690

@thornbill thornbill added bug Something isn't working stable backport Backport into the next stable release labels Jun 25, 2024
@thornbill thornbill added this to the v10.9.8 milestone Jun 25, 2024
@thornbill thornbill requested a review from a team as a code owner June 25, 2024 16:58
@thornbill thornbill merged commit 2d2d5be into jellyfin:release-10.9.z Jul 10, 2024
8 checks passed
@thornbill thornbill deleted the search-param-hook branch July 10, 2024 03:33
@jellyfin-bot jellyfin-bot removed the stable backport Backport into the next stable release label Jul 21, 2024
joshuaboniface pushed a commit that referenced this pull request Jul 21, 2024
Fix dashboard user page crash

Original-merge: 2d2d5be

Merged-by: thornbill <[email protected]>

Backported-by: Bill Thornton <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
No open projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants