-
Notifications
You must be signed in to change notification settings - Fork 2
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
Verify Account with Text-message or Email #164
Comments
using email or phone confirmation is ok with us. We do not want to collect phone numbers if the text verification holds onto that data. |
I would suggest the phone number option be a separate enhancement. It's an option I would choose myself for convenience, but adding the framework for SMS is a much larger project than leveraging the existing process for emails. That being said, I don't think it's a problem to ask for a phone number for verification and text it without permanently storing it. |
Database Changes
Code Changes
Tests
Email Confirmation Dialog box text (Rough Draft)
Email text (Rough Draft)Cara with LGROW to approve the final draft
Questions / Open Issues
|
I don't want the email to get too long, so let's go with this: (reuse header) Thanks , (first name), for your interest in adopting a drain in the Lower Grand River Watershed. Once you have clicked the button above, and your email is confirmed, please adopt a drain! (reuse signature) |
Problem
An individual can adopt an unlimited number of storm drains.
Solution
Complicate the new account process by adding a required response/confirmation before enable a new account. Confirmation sent to phone or email.
Tests
Add unit test to ensure the format of the text is correctAdd unit test to confirm the confirmation of the text messageThe text was updated successfully, but these errors were encountered: