Nico Domino

Sorry didn't mean to totally approve yet, just wanted to remove my one "requested changes" instance. Anyway, it does look pretty good already! I just committed a few small tweaks....

Followup Todos for us: - [ ] Add example repo (`nextauthjs/nuxt-example-app`) - [ ] Ensure Balazs github token can write to that (for sync) - [ ] Double check example...

Hey thanks for pointing this out! We've added some copy regarding client components, but nothing specifically regarding `useSession()` performance implications, I think thats a bit too "in the woods" for...

@buildbro hey thanks for your interest. We recently shipped a new docs site, so we need a docs page for the new provider at `docs/pages/getting-started/providers/authgear.mdx` as well 🙏

Want to do the example apps as well? They have their own `pnpm-lock.yaml`. Can be updated with `pnpm i --ignore-workspace` in their respective directories.

Hey thanks for the camelCase signIn related changes. Regarding the Entra import, that provider is a default export so you should be able to name it whatever you want. Can...

The stackoverflow answer you linked just checks if session token cookie is there or not, which might work in certain situations, but that session might be invalid, expired, etc. I...

Just curious why the current tests pass and why this isn't necessary for `createSession` or `createVerificationToken`, for example, then? :thinking:

@Sv3nskie it looks like your app is still using the pages router. But server components only exist in the app router paradigm. @rikurainio we added back an (updated) refresh token...

Replaced by: https://github.com/nextauthjs/next-auth/pull/11050