I'm trying to add email verification as documented...
# support-questions-legacyj
jacobluc
10/09/2022, 3:15 PMI'm trying to add email verification as documented here:
https://supertokens.com/docs/emailpassword/common-customizations/email-verification/about
However I get the following error:
Copy code
Argument of type '{ mode: string; }' is not assignable to parameter of type 'TypeInput'.jacobluc
10/09/2022, 3:15 PMr
rp_st
10/09/2022, 3:16 PMHey
j
jacobluc
10/09/2022, 3:16 PMhey 😄
jacobluc
10/09/2022, 3:16 PMI'm back with more silly questions 😉
r
rp_st
10/09/2022, 3:16 PMWhat’s the import statement? Also are you using the latest version of the SDKs?
j
jacobluc
10/09/2022, 3:17 PM Copy code
import EmailVerification from "supertokens-node/recipe/emailverification";jacobluc
10/09/2022, 3:17 PMis it a new feature? I have not updated the sdk since I started using it a couple of months back
r
rp_st
10/09/2022, 3:17 PMOh yea. New indeed
rp_st
10/09/2022, 3:18 PMIf you are updating it, I recommend you see the Changelog
j
jacobluc
10/09/2022, 3:18 PMdo you have. a link?
r
rp_st
10/09/2022, 3:19 PMrp_st
10/09/2022, 3:19 PMAnd also see the front end SDK Changelog
j
jacobluc
10/09/2022, 3:22 PMupdating got rid of the error
jacobluc
10/09/2022, 3:37 PMI have some trouble front-end now after updating to 0.26.4
jacobluc
10/09/2022, 3:38 PM Copy code
Type '(context: GetRedirectionURLContext) => void' is not assignable to type '(context: GetRedirectionURLContext) => Promise<string | undefined>'.
Type 'void' is not assignable to type 'Promise<string | undefined>'`jacobluc
10/09/2022, 3:38 PMI don't have to update a typescript @types package right?
r
rp_st
10/09/2022, 3:40 PMYou don’t have to update any types. It’s all part of the SDK
j
jacobluc
10/09/2022, 3:40 PMalright
jacobluc
10/09/2022, 3:40 PMi'm trying to follow the migration step found in the changelog
r
rp_st
10/09/2022, 3:40 PMYou should just return undefined at the end of the function instead of having no return
rp_st
10/09/2022, 3:41 PMI think that will solve the issue. But not too sure
j
jacobluc
10/09/2022, 3:41 PMjacobluc
10/09/2022, 3:41 PMNot sure if this is what you mean (line 40)
r
rp_st
10/09/2022, 3:42 PMMake the function async
j
jacobluc
10/09/2022, 3:42 PMyep
jacobluc
10/09/2022, 3:43 PMDo I need to have the preAPI and post APi hooks?
r
rp_st
10/09/2022, 3:43 PMyou don't need to add any of these if you don't need to add anything in them
j
jacobluc
10/09/2022, 3:43 PMsuper
jacobluc
10/09/2022, 3:46 PMDo I need to update the core docker image that I am hosting?
jacobluc
10/09/2022, 3:57 PMI got stuck in an infinite loop now, after a user has verified his email
jacobluc
10/09/2022, 3:57 PMthe page just keeps refreshing
jacobluc
10/09/2022, 4:07 PMI think it is the SessionAuth that is causing it
r
rp_st
10/09/2022, 4:59 PM> Do I need to update the core docker image that I am hosting?
If the backend SDK doesn't complain about it, you don't.
> I think it is the SessionAuth that is causing it
Hmm. Could be a lot of reasons. Would need more details
j
jacobluc
10/09/2022, 5:04 PMI just saw in the changelog that there had been an issue with it refreshing in loop
r
rp_st
10/09/2022, 5:05 PMHmmmm
j
jacobluc
10/09/2022, 5:10 PMdo you have time to look at it with me?
r
rp_st
10/09/2022, 5:10 PMNot today or tomorrow. But day after, yea
j
jacobluc
10/09/2022, 5:11 PMAlright 😄
r
rp_st
10/09/2022, 5:21 PMI suggest that you have a look at the quick setup docs again and see if there is something there that you haven’t done. Since I think the version of the SDK you used was pretty old
j
jacobluc
10/09/2022, 5:22 PMyeah, thats what i'm doing.
I think the problem is that I do not redirect after login now.
jacobluc
10/09/2022, 5:22 PMBut I'm a bit confused by the fact that the
r
rp_st
10/09/2022, 5:22 PMHmm. Well. It should redirect to / by default then
j
jacobluc
10/09/2022, 5:23 PM/ is where the login happens
jacobluc
10/09/2022, 5:23 PMThats where I have my login component
r
rp_st
10/09/2022, 5:23 PMAh ok. So then yea. You need to redirect to some other location
j
jacobluc
10/09/2022, 5:23 PMjacobluc
10/09/2022, 5:23 PMI'm a bit confused as to why the action can only be the string "VERIFY_EMAIL"
jacobluc
10/09/2022, 5:23 PMand does that mean that it went well, or that it is supposed to happen?
r
rp_st
10/09/2022, 5:24 PMOh. Is that in there email verification recipe?
j
jacobluc
10/09/2022, 5:24 PM/playerpage is where I want to redirect to
jacobluc
10/09/2022, 5:24 PMyes it is
jacobluc
10/09/2022, 5:24 PM Copy code
recipeList: [
EmailVerification.init({
mode: "REQUIRED",
// Props from emailVerificationFeature of the EmailPassword.init config should be moved here.
getRedirectionURL: async (context) => {
if (context.action === "VERIFY_EMAIL") {
return "/playerpage";
} else {
if (context.action === "SUCCESS") {
return "/playerpage" //TODO add course select page here
}
}
return undefined;
// Move handling email verification related redirects here
},
`r
rp_st
10/09/2022, 5:24 PMAh. So VERIFY_EMAIL means that it’s asking you where to redirect to verify the email.
rp_st
10/09/2022, 5:25 PMThe SUCCESS state still goes in the other recipe
rp_st
10/09/2022, 5:25 PMEmailpassword for example
j
jacobluc
10/09/2022, 5:25 PMahhh okay, I removed it from the emailpassword recipe
jacobluc
10/09/2022, 5:26 PMthat context.action no longer has a SIGN_IN_AND_UP string though
r
rp_st
10/09/2022, 5:27 PMYea. That’s moved to a top level function for redirection which gets TO_AUTH (I think)
rp_st
10/09/2022, 5:27 PMIt should be mentioned in the Changelog
j
jacobluc
10/09/2022, 5:27 PMyeah i saw something about it
jacobluc
10/09/2022, 5:27 PMMy issue is resolved now. The reason was that I kept redirecting
jacobluc
10/09/2022, 5:28 PMbecause I had removed the redirect from the emailPassword recipe
r
rp_st
10/09/2022, 5:28 PMYea fair!
j
jacobluc
10/09/2022, 5:29 PMI think I'm good to go now, thanks again for your help!