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

Adjust codegen logic for suspense patch #4138

Merged
merged 4 commits into from
May 1, 2023
Merged

Conversation

gjmoed
Copy link
Contributor

@gjmoed gjmoed commented Apr 30, 2023

Correctly account for Next 13.1 - 13.3.0
Add new for 13.3.1

(Yes, Blitz b26 depends on Next 13.3.0,
so we could account for >=13.3 scenarios only)

Closes: #4112

What are the changes and their implications?

Bug Checklist

  • Changeset added (run pnpm changeset in the root directory)
  • Integration test added (see test docs if needed)

Feature Checklist

Correctly account for Next 13.1 - 13.3.0
Add new for 13.3.1

(Yes, Blitz b26 depends on Next 13.3.0,
so we could account for >=13.3 scenarios only)

Fixes: blitz-js#4112
@gjmoed gjmoed requested a review from flybayer as a code owner April 30, 2023 06:52
@changeset-bot
Copy link

changeset-bot bot commented Apr 30, 2023

🦋 Changeset detected

Latest commit: 3f3cf70

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 8 packages
Name Type
@blitzjs/next Minor
next-blitz-auth Patch
@blitzjs/auth Minor
@blitzjs/rpc Minor
@blitzjs/codemod Minor
@blitzjs/config Minor
@blitzjs/generator Minor
blitz Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

Copy link
Member

@flybayer flybayer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thank you!!

@siddhsuresh siddhsuresh merged commit 3d004dc into blitz-js:main May 1, 2023
28 of 29 checks passed
@blitzjs-bot
Copy link
Contributor

Added @gjmoed contributions for doc and code

@blitzjs-bot blitzjs-bot added this to Done in Toolkit May 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Development

Successfully merging this pull request may close these issues.

A new app generated by blitz new causes Error: DYNAMIC_SERVER_USAGE
4 participants