Signin #7620
Signin #7620
Conversation
| @@ -0,0 +1,65 @@ | |||
| "use client"; | |||
There was a problem hiding this comment.
I would recommend calling the pages AttributesPage, CodePage, PasswordPage so that the client doesn't get confused where the code goes and how it's structured.
There was a problem hiding this comment.
@spetrescu84 the entry point of a route cannot be renamed in nextjs, it needs a page.tsx file for routes. Read here https://nextjs.org/docs/app/getting-started/layouts-and-pages#pages
There was a problem hiding this comment.
I see it can but it becomes too complex - https://nextjs.org/docs/pages/api-reference/config/next-config-js/pageExtensions. We can leave as is.
|
|
||
| const handleSubmitCode = async () => { | ||
| if (authState instanceof SignUpCodeRequired) { | ||
| const handler = AuthFlowStateHandlerFactory.create(authState); |
There was a problem hiding this comment.
Can't this one be moved inside the sdk? So there is an authState.handler which gets returned? Even if it's not used, it's a much better user experience to just say authState.handler.submitCode?
No description provided.