Skip to content

feat: implement client-side form validation using RHF and Zod - #50

Open
Divyansh77-cmd wants to merge 33 commits into
Areeb-coder:mainfrom
Divyansh77-cmd:feat/auth-validation
Open

Divyansh77-cmd wants to merge 33 commits into
Areeb-coder:mainfrom
Divyansh77-cmd:feat/auth-validation

Conversation

@Divyansh77-cmd

@Divyansh77-cmd Divyansh77-cmd commented May 23, 2026 •

Copy link
Copy Markdown
Contributor

🚀 Purpose
Implements robust client-side validation for Login and Signup pages to improve UX and reduce unnecessary API calls.

🛠️ Changes
Validation Schema: Created shared Zod rules in web/src/lib/validations.ts.

Login Page: Integrated react-hook-form and added real-time error states.

Signup Page: Added "Confirm Password" match logic and name validation.

UI: Applied Tailwind rose-500 error borders and disabled the submit button during loading.

🔒 Closes Issue
Closes #16

Screenshots:

Empty Fields / Login:
Fields missing

Email Format & Password Mismatch (signup page):
sign up page

Valid Form State:
all fields correct

Areeb-coder and others added 21 commits May 14, 2026 02:55
…olor-grading

fix: improve color contrast and fix washed out styling on homepage#2
…-hero

feat: add typewriter effect to hero headlines
Added dark/light mode theme variables, improved readability, smooth scrolling and accessibility-focused UI transitions.
This fix resolves a critical UX issue where mobile users were missing core navigation links in the header, leaving only the "Sign In" button. It introduces a responsive, fully animated hamburger menu dropdown that houses all section links and neatly integrates the "Sign In" CTA, restoring full site navigation for smaller viewports while preserving the platform's literary aesthetic.
- Integrated password visibility toggle button into the login and signup forms.
- Used Lucide React Eye and EyeOff icons for dynamic state indicators.
- Added accessibility labels (`aria-label`) for screen reader compatibility.
- Maintained existing responsive UI styling and layout continuity.
@vercel

vercel Bot commented May 23, 2026

Copy link
Copy Markdown

@Divyansh77-cmd is attempting to deploy a commit to the areeb-coder's projects Team on Vercel.

A member of the Team first needs to authorize it.

@Divyansh77-cmd

Copy link
Copy Markdown
Contributor Author

Hii @Areeb-coder
I have resolved the merge conflicts and fixed the TypeScript/JSX syntax errors in the login and signup pages. The PR is ready for review.

@Divyansh77-cmd

Copy link
Copy Markdown
Contributor Author

Hii @Areeb-coder
As told, I have successfully synced my branch with upstream/main locally, resolved all overlapping merge conflicts, and verified that both the login and signup validation layers are working cleanly with Zod and React Hook Form. This PR is ready for final review and merge!

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

ux: implement client-side form validation for auth routes

4 participants