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

Remove app router routing #6024

Merged
merged 1 commit into from
Sep 8, 2024
Merged

Conversation

thornbill
Copy link
Member

Changes
Removes the app router routing logic that has been (mostly) unused since adopting react router

Issues
N/A

@thornbill thornbill added the cleanup Cleanup of legacy code or code smells label Sep 5, 2024
Copy link

sonarqubecloud bot commented Sep 6, 2024

@jellyfin-bot
Copy link
Collaborator

Cloudflare Pages deployment

Latest commit 99b2bd4f6eddf7ecaa4cfb86beb3df7015b9ac8e
Status ✅ Deployed!
Preview URL https://0c43b77d.jellyfin-web.pages.dev
Type 🔀 Preview

@thornbill thornbill marked this pull request as ready for review September 6, 2024 21:51
@thornbill thornbill requested a review from a team as a code owner September 6, 2024 21:51
@thornbill thornbill added this to the v10.10.0 milestone Sep 8, 2024
@thornbill thornbill merged commit b37766e into jellyfin:master Sep 8, 2024
12 checks passed
@thornbill thornbill deleted the no-app-router branch September 8, 2024 01:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cleanup Cleanup of legacy code or code smells
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants