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

[User Registration] Reset Password Email Link Contains Incorrect URI #190

Open
BethanyG opened this issue Sep 24, 2020 · 5 comments
Open
Labels
bug Something isn't working help wanted Extra attention is needed P1 for MVP pinned

Comments

@BethanyG
Copy link
Member

Currently (as of PR #187 ) , when a user visits api/v1/auth/password/reset/ and POSTs their email address, a "password reset email" is sent containing a clickable URI that will send a user to a password reset form. The dev URI is incorrect, and will need to point to whichever React component on the front end that is going to handle password reset.

Current URI is:

http://django.codebuddies.org/api/v1/auth/password/reset/confirm/<UID>/<TOKEN>

but will need to be something pointing to the front end, but with the UID & TOKEN in it. The front-end will then need to POST to api/v1/auth/password/reset/confirm/ with the following:

{
    "new_password1": "",
    "new_password2": "",
    "uid": "",
    "token": ""
}

Which will reset the password for whichever user is represented by the UID.

@BethanyG BethanyG added bug Something isn't working help wanted Extra attention is needed P1 for MVP labels Sep 24, 2020
@BethanyG BethanyG changed the title [New User Registration] Reset Password Email Link Contains Incorrect URI [User Registration] Reset Password Email Link Contains Incorrect URI Sep 24, 2020
@stale
Copy link

stale bot commented Oct 24, 2020

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale label Oct 24, 2020
@BethanyG BethanyG removed the stale label Oct 24, 2020
@lpatmo
Copy link
Member

lpatmo commented Oct 24, 2020

Will update with the right URL when codebuddies/frontend#163 is merged in.

@stale
Copy link

stale bot commented Nov 23, 2020

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale label Nov 23, 2020
@BethanyG
Copy link
Member Author

Still open.

@stale stale bot removed the stale label Nov 23, 2020
@stale
Copy link

stale bot commented Dec 23, 2020

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale label Dec 23, 2020
@BethanyG BethanyG removed the stale label Dec 24, 2020
@lpatmo lpatmo added the pinned label Jan 19, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working help wanted Extra attention is needed P1 for MVP pinned
Projects
None yet
Development

No branches or pull requests

2 participants