search

KristerV / heliumpay-budgetweb-backend

2 stars 2 forks source link

Add user id to email links #21

Closed jschr closed 7 years ago

jschr commented 7 years ago

Need user id in the email links and also made emails slightly nicer.

KristerV commented 7 years ago

Isn't the token so unique that there is no need for userId? Or is it a optimization thing?

You can merge it either way. hmm how does the review thing work, can't find a way to access it :D

jschr commented 7 years ago

It's just because currently the route for updating includes the user id: users/:id/changePassword. The token is unique enough and in fact contains the user id but the frontend ideally shouldn't need to decode the token.