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

Chore - Add next 12 as peer dependency #45

Merged
merged 1 commit into from
Dec 4, 2022
Merged

Conversation

kiily
Copy link
Contributor

@kiily kiily commented Aug 12, 2022

Addressing the bug described in this issue: #39.

This should require no changes to how the module works but will simply allow it to be installed for users already on Next.js 12.

Let me know if this makes sense or whether a more in-depth explanation is needed🙏

@kiily
Copy link
Contributor Author

kiily commented Sep 13, 2022

Is there an update on when this would be merged or any further feedback? Let me know 🙏

@kiily
Copy link
Contributor Author

kiily commented Oct 18, 2022

Just a nudge on this one, we'd like to use next-safe in our project and this is the last remaining blocker

@I-keep-trying
Copy link

Is this going to work for Next13?

@kiily
Copy link
Contributor Author

kiily commented Nov 16, 2022

You're quite right! It will not... In principle, the code should work correctly in Next.js 13 but you won't be able to install it properly unless it is added here as a peer dependency 😢

Copy link
Owner

@trezy trezy left a comment

Choose a reason for hiding this comment

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

Great catch... I've deffo fallen behind here. I'll make sure to add Next 13 ASAP, as well.

@trezy trezy merged commit 79c6a2a into trezy:main Dec 4, 2022
@github-actions
Copy link

github-actions bot commented Dec 4, 2022

🎉 This PR is included in version 3.3.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants