Fixes issue introduced with pull request #7

mxstbr / login-flow

:key: A login/register flow built with React&Redux

https://loginflow.mxstbr.com

MIT License

1.6k stars 215 forks source link

Fixes issue introduced with pull request #7 #12

Closed jdm29 closed 8 years ago

jdm29 commented 8 years ago

pull request #7 introduced client side encryption of the password. This meant that a hashed value was being passed to auth.js even if a password is not entered. I have moved the field validation from auth.js to the AppActions which will work around this issue

mxstbr commented 8 years ago

@jdm29 why did you close this PR?

jdm29 commented 8 years ago

@mxstbr It was getting late and I'd made a mess of the commits. Should be ok now, please check

mxstbr commented 8 years ago

Validating that client side makes sense, thanks for this PR!