Rocket.chat: Changing email does not automatically resend verification email

[janrudolph]

Description:

Changing my email address should automatically send me a a verification email. As a result, I need to click the 'resend verification email' button. There is currently a bug that makes that resend button not work: for this see #7577

Server Setup Information:

• Version of Rocket.Chat Server: 0.59

Steps to Reproduce:

1. Change email address
2. Button occurs

Expected behavior:

The email should be sent automatically without user interaction.

Actual behavior:

The user has to click the resend button.

[abhishek71994]

So do you want the button removed and the email be sent based on some key event? As in if the user presses enter? or do you mean that as soon as the user presses save changes, then the email is sent?
@janrudolph

[janrudolph]

@abhishek71994 Right. The best way would be from my point of view:
If the user changes the email address and presses save, a verification link is sent to the new email address. Additionally a short message should appear which tells the user that a verification is required.

[abhishek71994]

@janrudolph I understand but what about the use case where the user enters a wrong email id? Can I connect with you on the Rocket.chat channel?

[janrudolph]

@abhishek71994 Sure, this is possible. However, the user is able to change the email right away to correct the wrong address?
sure, I am on open.rocketchat

[Joe-mcgee]

Hi, I'm new to the project and this seems like something a junior such as myself could tackle, have you guys already dug into it? or would it be free for me to take up

[abhishek71994]

@Joe-mcgee I'm working on this issue, kindly find another issue there are a lot of similar ones.

[abhishek71994]

@janrudolph Kindy check the PR

[janrudolph]

@abhishek71994 How have you implemented it now?

I thought about this issue yesterday:
I think we should send the verification mail right after the user has changed the mail address (as discussed above). However, we should also provide a manual button, in case the server has not sent the mail.

[abhishek71994]

@geekgonecrazy

[geekgonecrazy]

To re-iterate on the PR. Basically this PR removed the button. It needs to be there if the users email isn't verified. The only thing that should change is it should trigger the send email verification when the email address changes

[isabellarussell]

I have edited the description and issue title. Added labels too.

[rodrigok]

Closed by this commit