-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
[next] Fix edge runtime for route groups #9764
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Working great for me in prod!
… into ijjk/fix-edge-route-groups
Hi @ijjk - how can I use this in production? We've encountered this issue and with Next 13.2.x w/ app dir it won't work for me. Do I need to set the I would love to know what we can do to opt in b/c we're seeing slowness and I'd like to move our app dir project 100% to edge functions. |
@djfarrelly this should now be available in production, please re-deploy your project and give it a try! |
Thanks @ijjk! My latest deploy worked correctly now - I appreciate the help! |
This ensures we use the correct short name when generating edge functions with route groups from app directory.
Example deployment with fix can be seen here https://next-debug-edge-runtime-1ar7agbjh-vtest314-ijjk-testing.vercel.app/group-no-layout
Fixes: vercel/next.js#43458