samnabi / shopkit

Comprehensive commerce solution for Kirby CMS v2
https://shopkit.samnabi.com
Other
99 stars 15 forks source link

Password reset link error #141

Closed invasoria closed 7 years ago

invasoria commented 7 years ago

This happens on a fresh 1.1.6 install.

Use the form normally with a registered email: image

It returns an error: image

But, the reset link will still go through: image

After looking closely at the resetPassword() function, it seems to be missing a return to stop the error message from triggering. I added it like this (line 377):

image

Edit: I just noticed you already got that solved in your dev branch! Guess it'll be included soon in an upcoming version.

samnabi commented 7 years ago

Yes, I'm hoping to release 1.1.7 and 2.0 simultaneously... I just wanted to wait a bit in case there are other bugs that come up in the course of testing 2.0.

Thanks for reporting this!