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

add: with-next-auth-app-router example #180

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

dylsteck
Copy link

Adds examples/with-next-auth-app-router, a clone of the /examples/with-next-auth(now renamed with-next-auth-pages-router) repo that instead uses NextAuth.js 5 and the Next.js App Router. The new example repo features very minimal changes, but might serve as a helpful resource for other developers that want to use this stack in an app router repo.

Motivation

I was personally looking to use this stack for a project and wanted to put the example up in case others find it helpful for their integrations.

Change Summary

  • Renames /examples/with-next-auth to /examples/with-next-auth-pages-router
  • Creates a clone of the example above at /examples/with-next-auth-app-router, which uses the Next.js app router instead of the pages router and keeps most of the original code

Merge Checklist

Choose all relevant options below by adding an x now or at any time before submitting for review

  • PR title adheres to the conventional commits standard
  • PR has a changeset
  • PR has been tagged with a change label(s) (i.e. documentation, feature, bugfix, or chore)
  • PR includes documentation if necessary
  • All commits have been signed

Additional Context

If this is a relatively large or complex change, provide more details here that will help reviewers

also renames the old example to `with-next-auth-pages-router`
@deodad
Copy link
Member

deodad commented Jul 18, 2024

thanks @dylsteck!

@deodad deodad enabled auto-merge (squash) July 18, 2024 16:03
@deodad deodad disabled auto-merge July 18, 2024 16:03
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