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 SpaRouter #1784

Merged
merged 1 commit into from
Feb 25, 2023
Merged

Remove SpaRouter #1784

merged 1 commit into from
Feb 25, 2023

Conversation

davidpdrsn
Copy link
Member

ServeDir and ServeFile are easy enough to use that SpaRouter doesn't add much value.

`ServeDir` and `ServeFile` are easy enough to use that `SpaRouter`
doesn't add much value.
@davidpdrsn davidpdrsn added T-release Topic: releases breaking change A PR that makes a breaking change. A-axum-extra labels Feb 24, 2023
@davidpdrsn davidpdrsn removed the T-release Topic: releases label Feb 24, 2023
@davidpdrsn davidpdrsn merged commit 27f05ad into main Feb 25, 2023
@davidpdrsn davidpdrsn deleted the remove-spa-router branch February 25, 2023 10:05
@ZOXEXIVO
Copy link

ZOXEXIVO commented Mar 1, 2023

Not working as described in CHANGELOG:
Always return 404

@davidpdrsn
Copy link
Member Author

@ZOXEXIVO Please open an issue with a code sample instead of posting in closed PRs.

@kingsleyh
Copy link

SpaRouter was so easy to use - are there any examples of how support spa now that the SpaRouter was removed?

@kingsleyh
Copy link

nevermind found it in the changelog

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-axum-extra breaking change A PR that makes a breaking change.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants