added password_reset and password_reset_done templates - #2886
jamie-ferguson wants to merge 2 commits into
Conversation
|
Thanks Jamie, and sorry this sat so long. I checked it out locally and both pages render correctly inside the site chrome, the styling matches One thing before we merge: this covers the first half of the reset flow but not the second. The page a user lands on from the emailed link still falls through to allauth's own templates, with no site nav at all: So requesting a reset looks right, then clicking the link in the email drops you onto a bare allauth page to type the new password. The two still needed are:
They should be short, in the same shape as the two here. If you would like to finish it off, great. If you would rather not, say the word and we will add them and get this landed. |
c9aaf2f to
ab291c5
Compare
|
Hi, I've added the other two files requested and successfully tested them locally. This should be good to go now. Thanks. |
Description
Fixes missing password reset templates in the accounts pages.
Previously, attempting to reset a password would show unstyled pages at the following urls
/accounts/password/reset
/accounts/password/reset/done/
This PR adds two new template files which fixes this.
Tested locally using the Docker setup and triggering the password reset workflow.
Closes
Closes #2617