-
Notifications
You must be signed in to change notification settings - Fork 48
fix: update @opennextjs/aws dependency version to 756 #403
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
Conversation
|
commit: |
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.
Thanks!
I've put this as a draft for now. No need to merge this one until opennextjs/opennextjs-aws#725 is reviewed and merged. |
0756314
to
bbd41e6
Compare
packages/cloudflare/src/cli/build/open-next/createServerBundle.ts
Outdated
Show resolved
Hide resolved
Co-authored-by: Victor Berchet <victor@suumit.com>
Thanks for the review @vicb |
There has been a few changes in opennextjs/opennextjs-aws#725 to handle the node middleware.
It also includes changes for next 15.2 and page router data json not working opennextjs/opennextjs-aws#756
It changes the dependency on aws to "@opennextjs/aws": "https://pkg.pr.new/@opennextjs/aws@756"
This PR just add the necessary changes in cloudflare so that we could merge it in aws.
Technically the node middleware should just work in cloudflare but i haven't tested it (and i'm not sure if it even make sense)