Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Send registration confirmation email with verification link, remove u… #60

Merged
merged 8 commits into from
Sep 23, 2022

Conversation

ydahal1
Copy link
Collaborator

@ydahal1 ydahal1 commented Aug 22, 2022

…nverified accounts that are older than 12 hrs

ISSUE - #56

  1. Weather or not to verify user email, can be configured during the time of adding application
  2. If email settings are not added to back end envs, users won't have this option
  3. HPCC applications are set not to verify email
  4. Every 1 hr, a job runs and removes unverified users that are 12 hours or older
  5. Once user verifies the account, the confirmation code will be removed from users table and will be added to registration confirmation codes table
  6. If the user tries to verify account with link that is older than 12 hrs - they will get in-valid verification link message
  7. If user tries to verify already verified account, they get email already verified response
  8. Auth service default user have account verified flag true by default
  9. If a user uses roles are added for new application, no verification link will be sent
  10. Users can request to for new verification email

routes/auth.js Outdated Show resolved Hide resolved
routes/auth.js Outdated Show resolved Hide resolved
routes/notification.js Outdated Show resolved Hide resolved
Copy link
Collaborator

@konstagap konstagap left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

@konstagap konstagap merged commit 36d97e1 into dev Sep 23, 2022
@konstagap konstagap deleted the yadhap/registrationConfirmation branch September 23, 2022 18:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants