feat(fe): impl verification logic for whitelist removal#3552
Conversation
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request significantly expands the SignUpPage component, introducing features such as user ID and nickname availability checks, email verification with a timer, and dynamic selection for jobs, universities, and majors. It also integrates the korea-universities library and adds SKKU-specific registration logic. Feedback highlights a mismatch between password validation regex and its associated messages, suggests using the json option in API calls for consistency, and recommends adding error handling to the onSubmit function to improve the user experience.
|
✅ Syncing Preview App Succeeded Application: |
|
✅ Syncing Preview App Succeeded Application: |
|
✅ Syncing Preview App Succeeded Application: |
| import { signupSchema } from './signup.schema' | ||
| import type { SignUpFormValues } from './signup.type' | ||
|
|
||
| const JOB_OPTIONS = [ |
There was a problem hiding this comment.
5/6회의 기준) 고등학생 / 대학생 / 직장인 / 기타 4개를 직업선택지로 확정했습니다! 반영부탁드려용
| } | ||
| } | ||
|
|
||
| const generateNickname = () => { |
There was a problem hiding this comment.
기획안2 보시면 우아한 형제들에서 만든 한글 닉네임을 활용하기로 한걸 확인하실 수 있습니다! 그래서 해당 기획안보시고, 활용하시면 좋을거 같아요!
(링크: https://www.notion.so/skkuding/02-2adef9cff54580d2b143e1c1eae030e5)
*별명추천받는거 가능할까요?에 있는 댓글 참고하시면 됩니다
| placeholder="YYMMDD" | ||
| maxLength={6} | ||
| className={`placeholder:text-body1_m_16 h-[46px] w-full rounded-[12px] border bg-white px-5 py-[11px] outline-none placeholder:text-[#C4C4C4] ${ | ||
| className={cn( |
There was a problem hiding this comment.
생년월일 로직을 제거하기로 결정돼서 혹시 삭제해주실 수 있을까요??
| <div className="relative flex-1"> | ||
| <input | ||
| type="text" | ||
| placeholder="신나는 청사과" |
There was a problem hiding this comment.
기본 닉네임은 제가 임의로 작성했던거라, 우아한 형제들이 제공하는 기능을 활용하면 닉네임 placeholder를 그 중 하나로 변경하면 될거 같습니다!
|
[화면 기반 수정사항입니다!]
|
| type="button" | ||
| onClick={sendEmail} | ||
| disabled={!emailLocal || emailVerified} | ||
| className="text-sub3_sb_16 border-primary text-primary h-[46px] shrink-0 rounded-[12px] border px-4 disabled:cursor-not-allowed disabled:opacity-40" |
There was a problem hiding this comment.
화면상에서 '인증하기' 텍스트 크기가 잘 적용되지 않고 있는거 같습니다. '재발송'에 비해 작아보여서 확인부탁드립니다!
|
❗ Syncing Preview App Failed Application: |
|
main이 아니라 |
3 participants
Description
아이디 중복 확인, 이메일 인증, 닉네임 자동 생성, 직업 드롭다운, 대학교 검색 기능을 구현했습니다. 닉네임 중복 확인은 백엔드 API가 없어서 아직 구현하지 못했어요.
성균관대학교 선택 시 이메일 도메인이 @skku.edu로 고정되고 학과 검색과 학번 입력 필드가 추가됩니다. 그 외 대학교는 자유 이메일 입력이고 학과, 학번 항목은 표시되지 않습니다.
Additional context
closes TAS-2691
Before submitting the PR, please make sure you do the following
fixes #123).