DO NOT MREGE: Mfa/demo #1398
DO NOT MREGE: Mfa/demo #1398
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
How to use the Graphite Merge QueueAdd the label graphite-merge-queue to this PR to add it to the merge queue. You must have a Graphite account in order to use the merge queue. Sign up using this link. An organization admin has enabled the Graphite Merge Queue in this repository. Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue. |
| previousStep: "otp", | ||
| }); | ||
| } | ||
| }, [status]); |
There was a problem hiding this comment.
React Hook useEffect has a missing dependency: 'setAuthStep'. Either include it or remove the dependency array.
| constructor(params: BaseAlchemySignerParams<MockAlchemySignerWebClient>) { | ||
| super(params); | ||
| } |
There was a problem hiding this comment.
Useless constructor.
| ) { | ||
| setAuthStep({ type: "email_verify", email: authStep.email }); | ||
| } | ||
| }, [status]); |
There was a problem hiding this comment.
React Hook useEffect has missing dependencies: 'authStep.email', 'authStep.previousStep', and 'setAuthStep'. Either include them or remove the dependency array.
| ) { | ||
| setAuthStep({ type: "email_verify", email: form.state.values.email }); | ||
| } | ||
| }, [status]); |
There was a problem hiding this comment.
React Hook useEffect has missing dependencies: 'form.state.values.email' and 'setAuthStep'. Either include them or remove the dependency array.
Pull Request Checklist
yarn test)sitefolder, and guidelines for updating/adding docs can be found in the contribution guide)feat!: breaking change)yarn lint:check) and fix any issues? (yarn lint:write)PR-Codex overview
This PR focuses on implementing Multi-Factor Authentication (MFA) features across the application, enhancing security by adding support for TOTP (Time-Based One-Time Password) and related functionalities.
Detailed summary
useMFAhook for managing MFA operations.