Andrewnt219 / rent-near-me

https://rent-near-me.vercel.app/

1 stars 1 forks source link

centralized password validation #42

Closed Andrewnt219 closed 3 years ago

Andrewnt219 commented 3 years ago

Reason

I need the same validation for changing the password in account settings.

Note

Validation for containing first-name, last-name, email validation is removed because of the hassle when a form doesn't have those fields.
Avoid putting margin in reusable components. Margin should be assigned by layout-level components. space-y and space-x are helpful utilities.

vercel[bot] commented 3 years ago

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/andrewnt219/rent-near-me/BKvAWnGLdRNs3RM1zPKrR4h6tLV3
✅ Preview: https://rent-near-me-git-andrewnt219-centralize-pass-331812-andrewnt219.vercel.app

Andrewnt219 commented 3 years ago

I'll put the not contain fname, lname and email rule into the backlog for later. I think it's a good password security feature.

It's a great security feature indeed, but I don't think it's worth the effort to accommodate that.