-
Notifications
You must be signed in to change notification settings - Fork 12
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
i18nMiddleware TS error #34
Comments
Thanks @MarvinXu! I’ve been planning to use the new integration middleware feature so that you don’t have to configure the middleware at all—just haven’t gotten around to it quite yet. I decided to try it first by making cors and helmet middleware for Astro, and bringing back what I’ve learned to As for the new i18n feature, I’m disappointed that it doesn’t support what this library can—rendering all locales, including an unprefixed default locale, using a single I’ll keep this issue open until I make a PR for the built-in integration middleware. |
ha, exactly the reason I chose this integration :)
This looks great! Looking forward to it~ |
Fixed in Just remove the old …and everything will work the same! |
I encourage you to leave a comment and feedback in the i18n routing RFC 😄 I’d love to see Astro support it in core, or at least fix withastro/astro#8241 |
In new projects,
middleware.ts
shows an typescript error:I think it's because Astro updated types in the new release of i18n feature. Speaking of that, what's your opinion on the new i18n feature?
The text was updated successfully, but these errors were encountered: