Skip to content
This repository has been archived by the owner on Nov 22, 2023. It is now read-only.

Commit

Permalink
(PC-6976) : fix password reset link in email for native app
Browse files Browse the repository at this point in the history
when switching to the universal links, the part that was used to
detect the url redirection should have been removed but was not :(
  • Loading branch information
xordoquy committed Feb 17, 2021
1 parent cc12af1 commit d15e6fc
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion src/pcapi/emails/user_reset_password.py
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@ def retrieve_data_for_reset_password_native_app_email(
"email": user_email,
}
)
reset_password_link = f"{settings.NATIVE_APP_URL}/native/v1/redirect_to_native/mot-de-passe-perdu?{query_string}"
reset_password_link = f"{settings.NATIVE_APP_URL}/mot-de-passe-perdu?{query_string}"

return {
"MJ-TemplateID": 1838526,
Expand Down
2 changes: 1 addition & 1 deletion tests/emails/user_reset_password_test.py
Original file line number Diff line number Diff line change
Expand Up @@ -46,7 +46,7 @@ def test_email_is_encoded_in_(self, app):
"MJ-TemplateLanguage": True,
"Vars": {
"native_app_link": (
"https://app.passculture-testing.beta.gouv.fr/native/v1/redirect_to_native/mot-de-passe-perdu"
"https://app.passculture-testing.beta.gouv.fr/mot-de-passe-perdu"
"?token=abc&expiration_timestamp=1577836800&email=ewing%2Bdemo%40example.com"
)
},
Expand Down

0 comments on commit d15e6fc

Please sign in to comment.