Skip to content

Verify Your Account Today!#435

Merged
nsdeschenes merged 25 commits into
masterfrom
api-email-account-verification
May 28, 2020
Merged

Verify Your Account Today!#435
nsdeschenes merged 25 commits into
masterfrom
api-email-account-verification

Conversation

@nsdeschenes

Copy link
Copy Markdown
Contributor

Email verification on user sign-up is a go 🚀 but if that fails there is a backup mutation that will allow users to request another email be sent.

  • New email_validated field on the Users model, and new hybrid method verify_account
  • Functionality added to signUp mutation that will send a verification email
  • sendEmailVerification mutation allows the users to request another verification email be sent
  • emailVerifyAccount mutation allows users to verify their account through a token link sent to their inbox
  • User sandbox org is created once the user has verified their account now
  • Tests for all new functionality included
  • Tests corrected for new verify_account when querying their own sandbox org
  • frontend/schema.faker.graphql has been updated to reflect new changes
  • Minor changes and fixes

@sleepycat sleepycat left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

A few small changes, but this is looking great!

Comment thread api/db.py Outdated
Comment thread api/tests/test_organization_resolver_access_control.py Outdated
Comment thread api/functions/verification_email.py Outdated
Comment thread api/functions/verification_email.py Outdated
Comment thread api/models/Users.py Outdated
Comment thread api/db.py Outdated

@tparrott-cse tparrott-cse left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Verification a go go!

IdezHD added 22 commits May 28, 2020 09:46
…ication email, and made sleep time 1sec then check if email was successful
… the account creation, and moved email_verification function into functions directory
@nsdeschenes nsdeschenes force-pushed the api-email-account-verification branch from 699df8a to 18e6744 Compare May 28, 2020 12:48
@nsdeschenes nsdeschenes merged commit 0026d04 into master May 28, 2020
@nsdeschenes nsdeschenes deleted the api-email-account-verification branch May 28, 2020 12:54
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.

4 participants