-
Notifications
You must be signed in to change notification settings - Fork 35
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(codemod): UseSession Hook Update ( nextAuth - migration to v4) #1159
Conversation
@manishjha-04 is attempting to deploy a commit to the Codemod Team on Vercel. A member of the Team first needs to authorize it. |
@@ -0,0 +1,15 @@ | |||
{ | |||
"$schema": "https://codemod-utils.s3.us-west-1.amazonaws.com/configuration_schema.json", | |||
"name": "nextAuth/v4/useSession-array-to-object-destructuring", |
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.
Lets change the name to nextAuth/4/useSession-array-to-object-destructuring
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.
Sure✌️
"arguments": [], | ||
"meta": { | ||
"tags": ["nextAuth"], | ||
"git": "https://github.com/codemod-com/codemod" |
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.
Lets update the git
field to https://github.com/codemod-com/codemod/tree/main/packages/codemods/nextAuth/useSession-array-to-Object-destrucutring
Hi @manishjha-04 Thanks for your PR! Could you please rebase your branch on the latest |
@DmytroHryshyn Done👍.... Please review it and let me know if any changes needed! |
@manishjha-04 Thanks! Last 2 things: linter fails because of incorrect codemod files formatting (can be fixed by |
📚The useSession hook has been updated to return an object. This allows you to test states much more cleanly with the new status option.
🔗 Linked Issue #966
🧪 UseCase