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

feat: add email registration function #221

Closed
wants to merge 2 commits into from
Closed

feat: add email registration function #221

wants to merge 2 commits into from

Conversation

luhaoling
Copy link
Contributor

@luhaoling luhaoling commented Oct 9, 2023


🔍 What type of PR is this?

This PR is added to support email registration.

👀 What this PR does / why we need it:

  • My pull request adheres to the code style of this project
  • My code requires changes to the documentation
  • I have updated the documentation as required
  • All the tests have passed

🅰 Which issue(s) this PR fixes:

#222

📝 Special notes for your reviewer:

When you want to use other maibox to send email,you just to update the fileds "sendMail senderAuthorizationCode smtpAddr smtpPort" of the config/config.yaml.

🎯 Describe how to verify it

When you invoke the send verification api, fill in the email and leave the mobile number field blank.

📑 Additional documentation e.g., RFC, notion, Google docs, usage docs, etc.:

None.

@luhaoling luhaoling requested review from cubxxw and a team as code owners October 9, 2023 02:40
@pull-request-size pull-request-size bot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Oct 9, 2023
@kubbot
Copy link
Contributor

kubbot commented Oct 9, 2023

CLA Assistant Lite bot: 💕 Thank you for your contribution and please kindly read and sign our 🎯https://github.com/openim-sigs/cla/blob/main/README.md.
If you wish to sign the CRA, Please copy and comment on the following sentence:


I have read the CLA Document and I hereby sign the CLA


You can retrigger this bot by commenting recheck in this Pull Request

Copy link
Contributor

cubxxw commented Oct 9, 2023

You need to specify an issue

@luhaoling
Copy link
Contributor Author

You need to specify an issue

OK,I understand.

@luhaoling
Copy link
Contributor Author

I have read the CLA Document and I hereby sign the CLA

@luhaoling
Copy link
Contributor Author

You need to specify an issue

I hava specified an issue.

@luhaoling luhaoling closed this by deleting the head repository Oct 12, 2023
@luhaoling luhaoling mentioned this pull request Oct 12, 2023
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size/L Denotes a PR that changes 100-499 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants