Skip to content
This repository has been archived by the owner on Jan 26, 2021. It is now read-only.

feat:send email verification for sign up #1168

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from
Open

feat:send email verification for sign up #1168

wants to merge 2 commits into from

Conversation

harsh7in
Copy link

Description

Changes made for verification when the user signup

Fixes #1029

Type of Change:

  • Code
  • Quality Assurance

Code/Quality Assurance Only

  • New feature (Send email to the user after signup)

Checklist:

  • My PR follows the style guidelines of this project
  • I have performed a self-review of my own code or materials

Code/Quality Assurance Only

  • My changes generate no new warnings

@harsh7in
Copy link
Author

@keshakaneria ma'am can you plz review this I don't know why my tests are failing and do it need to change .env.example file?

@keshakaneria
Copy link
Member

@keshakaneria ma'am can you plz review this I don't know why my tests are failing and do it need to change .env.example file?

Have a look at this line. You might have altered the configuration.

@keshakaneria
Copy link
Member

@harsh-9in the error still remains there. Have a look at that same error again.

@harsh7in
Copy link
Author

@harsh-9in the error still remains there. Have a look at that same error again.

Yes I am trying to resolve it

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add task using Celery to send email verification for signing up
2 participants