infisical icon indicating copy to clipboard operation
infisical copied to clipboard

Automatically validate the email verification code (no need to press <ENTER>)

Open Grraahaam opened this issue 11 months ago • 11 comments

Feature description

Today, when I create an account, I'm receiving the verification code to sign in by email. I then need to copy and paste it in the app, after what I need to press ENTER or the submit button.

  1. We could validate the code directly when pasting it in the input, or once all the fields have been populated?
  2. I've seen a topic somewhere here about replacing the verification code by a link to simplify this process

What do you think?

Why would it be useful?

It'd improve the UX and simplify the user's verification process

Additional context

Related issues:

  • https://github.com/Infisical/infisical/issues/58
  • https://github.com/Infisical/infisical/pull/683
  • https://github.com/Infisical/infisical/pull/1529
  • https://github.com/Infisical/infisical/issues/555

Grraahaam avatar Mar 17 '24 19:03 Grraahaam

@Grraahaam I can work on this issue if anyone else is not working currently

kavinda1995 avatar Mar 18 '24 03:03 kavinda1995

I agree, we could improve the UX by letting it automatically continue after pasting the full code.

@kavinda1995 Yes! That would be awesome!

DanielHougaard avatar Mar 18 '24 08:03 DanielHougaard

@DanielHougaard This change doesnt change the validation logic itself, so it's easy to implement and will already help.

But I think that the verification link would be an even better idea in the long term (e.g. code already in the URL) as it'll make the verification step even faster and less prone to error for people who wrongly copy/pasted the code from the mail for example (e.g. missing digits or mail formatting issues), but the logic might need to be updated in this case.

WDYT?

Grraahaam avatar Mar 18 '24 10:03 Grraahaam

@Grraahaam I agree, that would be even better. I wouldn't start doing that just yet though. We need to make the emails nicer in general at some point, and I think such a change would fall pretty nicely into that scope.

For now I think making it automatically continue efter entering the code is the right move!

DanielHougaard avatar Mar 18 '24 11:03 DanielHougaard

For now I think making it automatically continue efter entering the code is the right move!

Absolutely!

We need to make the emails nicer in general at some point, and I think such a change would fall pretty nicely into that scope.

Actually I've started an email branding branch a while ago (https://github.com/Infisical/infisical/pull/444), but didn't took time to migrate it to https://github.com/resendlabs/react-email

Grraahaam avatar Mar 18 '24 11:03 Grraahaam

Oh wow I didn't see that one, awesome! React email is great, and this is actually what I had in mind for the mail refactor. Currently it's not a huge priority, but it will defiantly become a more pressing topic in the future.

DanielHougaard avatar Mar 18 '24 11:03 DanielHougaard

Is this issue resolved?

vishavsingla avatar Mar 24 '24 09:03 vishavsingla

Hey @vishavsingla ! Doesnt seems to be..

@kavinda1995 was willing to work on it

Grraahaam avatar Mar 24 '24 14:03 Grraahaam

Is the issue resolved?

siddhant-agrawal01 avatar Jun 26 '24 16:06 siddhant-agrawal01

I'll take this if it's available

akshat99812 avatar Jun 28 '24 15:06 akshat99812

@Grraahaam I can work on this issue if issue is open

rahuljagtapp avatar Jul 22 '24 09:07 rahuljagtapp

@Grraahaam Can you please assign me this issue, if it is still open

saloni0419 avatar Oct 08 '24 21:10 saloni0419