-
-
Notifications
You must be signed in to change notification settings - Fork 165
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
feat: getSession force refetch option #425
feat: getSession force refetch option #425
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for this contribution ❤️ Please also extend the docs with the new config option and we are good to go (:
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Almost there:
- I should've let you know: You have to change the docs of v0.6 as this is the version that will receive this feature. They are located at
docs/v0.6/
- can you move your changes there? - your changes only effect the
local
-provider - when you move your doc-changes to thedocs/v0.6
part, it should also be clearer, that this is the case
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Adds a
force
option togetSession
. Useful when token cookie has been updated by an external api route (not handled by nuxt-auth'ssignIn
/signOut
) because the token inuseAuthState
would still beundefined
/null
after manually calling a register api route for example.