develop-me / bagajob

Job Hunting Management App built by Develop Me students for Develop Me students.
https://bagajob.developme.space
GNU Affero General Public License v3.0
1 stars 2 forks source link

Password reset - no route exists for URL supplied in email #46

Closed kierancott closed 3 years ago

kierancott commented 4 years ago

Issue: The URL supplied in the email is not hitting a route. e.g.: http://localhost:3000/password-reset/?email=nik%40nikolaso.com&token=fR7oRdqYuCgPms2N2QzSxMMCZsGX9aO8hke9l4bFGbqpbn1YIYnwn0jE5lUc where email=nik%40nikolaso.com & token=fR7oRdqYuCgPms2N2QzSxMMCZsGX9aO8hke9l4bFGbqpbn1YIYnwn0jE5lUc are query parameters.

Action: Modify the password-reset route in App.js from /password-reset/:token to handle both the email and token query parameters as above.

kierancott commented 4 years ago

Updated

Commit: https://github.com/develop-me/bagajob/commit/55630843d1a5c240161331762018bed7dedafe9a

Action:

Result:

Screenshot 2020-09-21 at 15 33 19
kierancott commented 3 years ago

Updated

Email, Token and Password now being sent to server with status code 200.

Screenshot 2020-09-22 at 12 24 21 Screenshot 2020-09-22 at 12 25 10
nosvalds commented 3 years ago

I'm seeing this resolved in my testing!