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

Nextjs improvements #29

Merged
merged 3 commits into from
Oct 26, 2023
Merged

Nextjs improvements #29

merged 3 commits into from
Oct 26, 2023

Conversation

Billlynch
Copy link
Collaborator

@Billlynch Billlynch commented Oct 13, 2023

Hi There, I just made a Pull Request!

Add documentation and example apps for usage in Next 12 and Next 13.

Add use client directive to the React Adapter to tell Next13 to include in the client bundle and warn about server side use.

✔️ Checklist

  • All tests are passing
  • Relevant documentation updated
  • linter/style run on changed files
  • Tests added for new functionality
  • Regression tests added for bug fixes

@Billlynch Billlynch force-pushed the nextjs-improvements branch 2 times, most recently from ed6db88 to 21ca564 Compare October 26, 2023 15:03
Copy link
Contributor

@andreas-karlsson andreas-karlsson left a comment

Choose a reason for hiding this comment

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

LGTM!

@Billlynch Billlynch merged commit 151015a into main Oct 26, 2023
3 checks passed
@Billlynch Billlynch deleted the nextjs-improvements branch October 26, 2023 15:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants