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

Add feature to send OTPs in e-mail verifications. #776

Merged
merged 7 commits into from
Oct 26, 2023

Conversation

shanggeeth
Copy link
Contributor

This is implemented as fix to the wso2/product-is#14808
Applying changes from #617

Following are the changes introducing through this PR.

  1. Checkbox (Send OTP in E-mail) to configure whether to send a confirmation code or a OTP in verification/confirmation e-mail.
  2. Three checkboxes (Use uppercase characters in OTP, Use lowercase characters in OTP, User numbers in OTP) to define OTP pattern for e-mail and SMS OTPs.
  3. Input field to include the length for the OTPs.

These changes are introduced to the below flows.

  1. Self Registration
  2. Lite Registration
  3. Ask Password
  4. User Claim Update.
  5. Notification based Password Recovery

@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 2 committers have signed the CLA.

✅ shanggeeth
❌ DilshaniSH
You have signed the CLA already but the status is still pending? Let us recheck it.

@jenkins-is-staging
Copy link

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/6570076862

@jenkins-is-staging
Copy link

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/6570076862
Status: failure

@jenkins-is-staging
Copy link

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/6651396232

@jenkins-is-staging
Copy link

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/6651396232
Status: failure

@shanggeeth
Copy link
Contributor Author

The failure is expected due to these changes and is fixed with

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.

7 participants