search

rafinkanisa / ngm-reportDesk

The workdesk for ReportHub
0 stars 1 forks source link

Deactivated account still listed in the Forgot Password email #148

Closed rafinkanisa closed 2 years ago

rafinkanisa commented 3 years ago

If you go to the Forgot Password page and submit an email

If this email is associated with several accounts and some of the account is already deactivated the email that being sent to reset the password is including the deactivated account

I think we should only send the link for username which in the active state What do you think?

fakhrihawari commented 3 years ago

I prefer we still show it but with some information like this

Screen Shot 2021-10-29 at 15 16 20
rafinkanisa commented 3 years ago

That's a good idea

I think you could put the sentence as the following : Cannot reset password. This account has been DEACTIVATED.

fakhrihawari commented 2 years ago
Screen Shot 2021-11-18 at 14 24 04
fakhrihawari commented 2 years ago

Link PR: https://github.com/iMMAP/ngm-reportEngine/pull/27