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.

Sign up for GitHub

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

Jump to bottom

[BE] Mail reset password #82

Closed
CloudSalander opened this issue Sep 13, 2023 · 5 comments · Fixed by #108 or #103
Closed

[BE] Mail reset password #82

CloudSalander opened this issue Sep 13, 2023 · 5 comments · Fixed by #108 or #103
Assignees
@FranEnLaNube @brycoti

Comments

@CloudSalander
Copy link
Collaborator

No description provided.

@CloudSalander CloudSalander converted this from a draft issue Sep 13, 2023
@CloudSalander
Copy link
Collaborator Author

CloudSalander commented Sep 13, 2023 •
edited
Loading

  • Verify if magent its the same color than frontend.
  • Add explicit url for password reset on the email.
  • Generate token for add it to the react app.
  • Test mail sending in server.

@FranEnLaNube
Copy link
Collaborator

@CloudSalander Wasn't this Issue closed in #88?

@CloudSalander
Copy link
Collaborator Author

We have to update the hardcoded url of the reset-password link :)

Specifically to the domain https://it-academy-landing.netlify.app/

@CloudSalander
Copy link
Collaborator Author

CloudSalander commented Nov 29, 2023 •
edited
Loading

Okey, we don't know if actually exists in prod environment the reset-password view. We will wait to @vicenlu confirmation.

@CloudSalander CloudSalander moved this from 🆕 New to On hold in ita-landing Nov 29, 2023
@brycoti brycoti mentioned this issue Nov 29, 2023
Merged
@FranEnLaNube FranEnLaNube mentioned this issue Nov 29, 2023
Merged
@FranEnLaNube
Copy link
Collaborator

Great job @brycoti with your #103 to resolve #82! You are really welcomed in this project! 💯
Let me add my approach to attack this problem in #108 .
I hope is good!

@FranEnLaNube FranEnLaNube linked a pull request Nov 29, 2023 that will close this issue
[BE] Hotfix, Update reset password link in mail template #103
Merged
@github-project-automation github-project-automation bot moved this from On hold to ✅ Done in ita-landing Dec 25, 2023
SectionOne added a commit that referenced this issue Dec 25, 2023
@SectionOne
Merge pull request #108 from IT-Academy-BCN/hotfix/reset-password-link
6ddca3f 
Fix/Send $url (set in .env) to forgetPassword template from UserController::forgetPassword() #82
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@FranEnLaNube FranEnLaNube

@brycoti brycoti

Labels
None yet
Projects
ita-landing
Status: ✅ Done
Milestone
No milestone
3 participants
@CloudSalander @FranEnLaNube @brycoti