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

Update the next-auth readme to add details about pages router support + api reference #814

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

jaclarke
Copy link
Member

No description provided.

@diksipav
Copy link
Contributor

Hey @jaclarke I’m still a bit confused, the things we have in the client are methods on the client class, and then also on the server class so we call them with auth.getBuiltinUIUrl for example , I still don’t get a complete idea why would someone need these links without full functionality…

I wrote a readme, pls check it (I mostly copied the flow and info that you provided in next example).
@scotttrinh added more detailed info for Express https://github.com/edgedb/edgedb-js/blob/master/packages/auth-express/readme.md.

Should I also go into more details explaining (explaining error handling, that callbacks should preferably return response/redirect etc).

@scotttrinh scotttrinh marked this pull request as ready for review February 23, 2024 19:57
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.

None yet

2 participants