Skip to content

fix(clerk-js): Prevent post auth redirects in Metamask flow#31

Merged
SokratisVidros merged 1 commit intomainfrom
metamask_auth_fix
Feb 9, 2022
Merged

fix(clerk-js): Prevent post auth redirects in Metamask flow#31
SokratisVidros merged 1 commit intomainfrom
metamask_auth_fix

Conversation

@SokratisVidros
Copy link
Copy Markdown
Contributor

Type of change

  • 🐛 Bug fix
  • 🌟 New feature
  • 🔨 Breaking change
  • 📖 Refactoring / dependency upgrade / documentation
  • other:

Packages affected

  • @clerk/clerk-js
  • @clerk/clerk-react
  • @clerk/nextjs
  • @clerk/clerk-remix
  • @clerk/clerk-expo
  • @clerk/backend-core
  • @clerk/clerk-sdk-node
  • @clerk/edge
  • build/tooling/chore

Description

  • npm test runs as expected.
  • npm run build runs as expected.

Check if session is created before calling Clerk.setSession and redirecting the user.
Especially during sign ups, the session wont be created unless all the sign up requirements (e.g. names and emails) are met.

Check if session is created before calling Clerk.setSession and redirecting the user.

Especially during sign ups, the session wont be created unless all the sign up requirements (e.g. names and emails)  are met.
@SokratisVidros SokratisVidros merged commit 052ff1e into main Feb 9, 2022
@SokratisVidros SokratisVidros deleted the metamask_auth_fix branch February 9, 2022 14:25
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Jun 11, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants