Skip to content

feat: zod 기반 회원가입 실시간 유효성 검사 적용#160

Merged
chaeyeon-dev merged 2 commits into
developfrom
feat/signup-realtime-validation
Apr 17, 2026
Merged

feat: zod 기반 회원가입 실시간 유효성 검사 적용#160
chaeyeon-dev merged 2 commits into
developfrom
feat/signup-realtime-validation

Conversation

@chaeyeon-dev

Copy link
Copy Markdown
Contributor

무엇을 변경했나요?

zod 기반 회원가입 실시간 유효성 검사 적용

왜 이렇게 했나요?

버튼 클릭 후에만 에러가 표시되던 방식 -> 입력 중 즉시 피드백을 주는 방식

리뷰어가 특히 봐줬으면 하는 부분

  • (예: 이 로직이 엣지케이스를 잘 처리하는지)

스크린샷 (UI 변경 시)

@vercel

vercel Bot commented Apr 16, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
bearlog Ready Ready Preview, Comment Apr 17, 2026 0:31am

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request refactors the signup page to use react-hook-form and zod for form management and validation, replacing manual state handling. It also applies a specific text color to the submit buttons in both the login and signup flows. Feedback highlights that validation messages in the Zod schema should be internationalized using the t() function rather than being hardcoded. Additionally, it is recommended to move hardcoded hex colors into the Tailwind theme configuration to ensure design consistency and improve maintainability.

Comment thread src/app/(auth)/signup/page.tsx Outdated
Comment thread src/app/(auth)/login/page.tsx
Comment thread src/app/(auth)/signup/page.tsx Outdated

@join0life join0life left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

오 적용하셨군요!

고생 많으셨습니다 👍

@chaeyeon-dev chaeyeon-dev merged commit 22eafa3 into develop Apr 17, 2026
5 checks passed
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.

2 participants