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

#166240990 Send a verification link after a successful registration #20

Merged
merged 1 commit into from
Jun 14, 2019

Conversation

eliemugenzi
Copy link
Contributor

@eliemugenzi eliemugenzi commented Jun 12, 2019

What does this PR do?

Send a verification link via email after a successful registration

Description of Task to be completed?

  • User signs up
  • A server validates the information
  • If yes, it registers a user
  • Send a verification link to the new user's email
  • If a user clicks a link from an email, a server checks the user
  • If the user is valid, VERIFICATION IS DONE

How should this be manually tested?

Clone the repository, this branch

  • yarn install
  • Run sequelize db:migrate to run the migrations
  • Run yarn dev to run the server
  • Hit the endpoint /api/auth/signup , fill out the data and send the request. Use POST request.
  • It will send an email, click click here to verify your account
  • It will take you to your default browser and you can get a response whether your account is verified or not.

Any background context you want to provide?

N/A

What are the relevant pivotal tracker stories?

#166240990

Screenshots (if appropriate)

Screen Shot 2019-06-12 at 15 40 42

Questions:

N/A

src/sequelize/models/user.js Outdated Show resolved Hide resolved
src/sequelize/migrations/20190612104330-create-user.js Outdated Show resolved Hide resolved
src/sequelize/migrations/20190612104330-create-user.js Outdated Show resolved Hide resolved
src/sequelize/migrations/20190612104330-create-user.js Outdated Show resolved Hide resolved
src/sequelize/migrations/20190612104330-create-user.js Outdated Show resolved Hide resolved
src/api/controllers/auth.js Outdated Show resolved Hide resolved
src/api/controllers/auth.js Outdated Show resolved Hide resolved
src/api/controllers/auth.js Outdated Show resolved Hide resolved
src/api/controllers/auth.js Outdated Show resolved Hide resolved
@eliemugenzi eliemugenzi force-pushed the ft-verify-account-166240990 branch 4 times, most recently from 1bd5e43 to d2e9e6b Compare June 12, 2019 16:58
@eliemugenzi eliemugenzi force-pushed the ft-verify-account-166240990 branch 10 times, most recently from ea39b5d to f6ec0ea Compare June 13, 2019 16:19
test/auth.js Outdated Show resolved Hide resolved
test/auth.js Outdated Show resolved Hide resolved
src/api/controllers/auth.js Outdated Show resolved Hide resolved
src/api/controllers/auth.js Outdated Show resolved Hide resolved
@eliemugenzi eliemugenzi added Needs Merge Needs To merge Needs Review and removed Needs Merge Needs To merge WIP labels Jun 13, 2019
src/api/controllers/auth.js Outdated Show resolved Hide resolved
src/api/controllers/auth.js Outdated Show resolved Hide resolved
src/config/environments.js Outdated Show resolved Hide resolved
@eliemugenzi eliemugenzi force-pushed the ft-verify-account-166240990 branch 4 times, most recently from 512a365 to 343ec64 Compare June 13, 2019 18:50
@eliemugenzi eliemugenzi force-pushed the ft-verify-account-166240990 branch 3 times, most recently from 16b8f8f to 5b0897e Compare June 13, 2019 19:29
test/auth.test.js Outdated Show resolved Hide resolved
@eliemugenzi eliemugenzi force-pushed the ft-verify-account-166240990 branch 4 times, most recently from d6680a0 to 587bbc0 Compare June 13, 2019 19:50
.env.example Outdated Show resolved Hide resolved
.gitignore Outdated Show resolved Hide resolved
.travis.yml Outdated Show resolved Hide resolved
src/db/index.js Outdated Show resolved Hide resolved
src/helpers/Env.helper.js Outdated Show resolved Hide resolved
src/helpers/signupSchema.js Outdated Show resolved Hide resolved
@eliemugenzi eliemugenzi force-pushed the ft-verify-account-166240990 branch 2 times, most recently from 14bf01e to 8d0b4ec Compare June 14, 2019 06:30
@eliemugenzi eliemugenzi added Needs Merge Needs To merge and removed Needs Review labels Jun 14, 2019
@Quantum-35 Quantum-35 merged commit e1cfd60 into develop Jun 14, 2019
eliemugenzi pushed a commit that referenced this pull request Jun 19, 2019
feat: add registration validation errors (#18)

* feat: add registration validation errors

* feat(profiles): implement profiles functionality

- implement user profile creation
- implement user profile fetching functionality
- document endpoints
- fix token middleware
- make request validation middleware reusable

[Finishes #166240995]

feature(project) Send an account verification link via email (#20)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Needs Merge Needs To merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants