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

Clarify Next.js example #5

Closed
wants to merge 1 commit into from
Closed

Clarify Next.js example #5

wants to merge 1 commit into from

Conversation

leerob
Copy link

@leerob leerob commented Nov 1, 2023

Thanks for updating with that new example! We can help review that one to make sure it's showing the correct patterns. It's already much better than the previously very outdated one which I would not recommend. I don't think it's worth including there here, as it will start folks on the wrong foot and not represent Next.js well.

@addyosmani
Copy link
Member

Thanks @leerob! I've been working on a rework of the landing page (which included this older version). I'll go ahead and drop it from the references list to align with your PR shortly.

addyosmani added a commit that referenced this pull request Nov 3, 2023
This drops the older Next 12.x version. It's primary deltas to the current more up to date version are that it features authentication and more features.
@addyosmani
Copy link
Member

Closed in 3c3b3a9

@addyosmani addyosmani closed this Nov 3, 2023
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