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

Email sent to user for password verification #1030 #1115

Conversation

kritikaparmar-programmer

Description

Now the email is sending to the user for password reset

Fixes #1030

Type of Change:

Change in settings.py

  • Code

Code/Quality Assurance Only

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality pre-approved by mentors)

Checklist:

Delete irrelevant options.

  • 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

@kritikaparmar-programmer
Copy link
Author

kritikaparmar-programmer commented Sep 13, 2020

@keshakaneria Take a look at the changes now..

Copy link

@SanketDG SanketDG left a comment

Choose a reason for hiding this comment

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

I must be missing something, but was anything mentioned in the issue implemented at all? 🤔

@kritikaparmar-programmer
Copy link
Author

@keshakaneria asked me to do what I know to send email to the user for forgot password..
And I told her that I 'll not use Celery
@SanketDG

@SanketDG
Copy link

But putting actual/real settings in the settings file does not solve the actual problem, right?

I must be missing something here, apologies.

@keshakaneria
Copy link
Member

But putting actual/real settings in the settings file does not solve the actual problem, right?

I must be missing something here, apologies.

Yes, I thought the things which she has done earlier might be compatible to resolve this problem. But i guess we will have to look into it @kritikaparmar-programmer. Can you do it with celery as defined in the issue?

@kritikaparmar-programmer
Copy link
Author

@keshakaneria Please assign this issue to someone else.. I tried but not able to do..

@keshakaneria
Copy link
Member

@keshakaneria Please assign this issue to someone else.. I tried but not able to do..

Alright :)

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 reset password
3 participants