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: |
| <div className="relative flex-1"> | ||
| <input | ||
| type="text" | ||
| placeholder="신나는 청사과" |
There was a problem hiding this comment.
기본 닉네임은 제가 임의로 작성했던거라, 우아한 형제들이 제공하는 기능을 활용하면 닉네임 placeholder를 그 중 하나로 변경하면 될거 같습니다!
|
[화면 기반 수정사항입니다!]
|
|
❗ Syncing Preview App Failed Application: |
|
main이 아니라 |
|
✅ Syncing Preview App Succeeded Application: |
|
✅ Syncing Preview App Succeeded Application: |
|
korean-random-names-generator 닉네임 생성 패키지를 설치하고 placeholder를 변경했습니다. 아래는 지난 회의 노션에 기록된 기획 관련 수정 사항입니다.
|
|
❗ Syncing Preview App Failed Application: |
|
✅ Syncing Preview App Succeeded Application: |
@nhjbest22 stage 백엔드가 아직 main 기준으로 배포되어 구버전 코드가 실행 중이어서 Preview 페이지와 로컬 환경에서 외부 이메일로 전송이 실패하고 있습니다. 프론트엔드 테스트 목적으로 해당 백엔드 PR만이라도 먼저 main에 머지하는 게 가능할까요? |
|
✅ Syncing Preview App Succeeded Application: |
59c73ef to
f308a97
Compare
|
✅ Syncing Preview App Succeeded Application: |
|
✅ Syncing Preview App Succeeded Application: |
|
✅ Syncing Preview App Succeeded Application: |
|
✅ Syncing Preview App Succeeded Application: |
|
✅ Syncing Preview App Succeeded Application: |
|
✅ Syncing Preview App Succeeded Application: |
->저번에 달았던 리뷰인데, border 삭제(2번째)는 잘 작동합니다!! 혹시 1번째 내용인 '아이디 중복 확인을 해주세요'가 1초 나오고, '사용 가능한 아이디입니다'가 보이는데, 한번 확인해주실 수 있으실까용? |
|
✅ Syncing Preview App Succeeded Application: |
|
✅ Syncing Preview App Succeeded Application: |
넵 수정했습니다! 다시 확인 부탁드려요! |
### Description <!-- Please insert your description here and provide especially info about the "what" this PR is solving --> <img width="1894" height="1482" alt="image" src="https://github.com/user-attachments/assets/7b40440e-8e64-4efb-905d-a5aa059fb834" /> <img width="1858" height="1482" alt="image" src="https://github.com/user-attachments/assets/b3ef69fa-5393-4fb7-a3b1-77efee402ba1" /> <img width="864" height="640" alt="image" src="https://github.com/user-attachments/assets/ac0e0845-8bd5-4364-b869-620bf6f5b569" /> - 닉네임 (수정 가능) 0자 이상 20자 이하입니다. 닉네임이 없으면 빈 칸으로 표시되고, 실제 API 데이터가 오면 기존 닉네임으로 채워집니다. 닉네임이 기존값과 달라지면 저장 버튼이 활성화됩니다. - 직업 (현재 수정 불가) 화이트리스트 제거 이전에 가입한 기존 회원은 jobType이 null로 저장되어 있어 설정 페이지에서 "미설정"으로 표시됩니다. 이 사용자들이 직접 직업을 설정할 수 있도록 수정이 필요합니다. (논의 필요) - 대학교 (수정 가능) 문제점: 타 대학 ↔ 성균관대로 변경할 경우 @skku.edu 이메일 사용 여부 문제가 있습니다. 성균관대 → 타 대학의 경우, isSKKU가 studentId !== '0000000000'으로 결정되기 때문에 SKKU 학번이 있는 사용자는 설정 페이지에서 대학교 필드 자체가 읽기 전용으로 표시됩니다. 이쪽은 이미 막혀 있습니다. 타 대학 → 성균관대의 경우, 대학교를 성균관대학교로 저장하면 SKKU 학생처럼 동작하는 문제가 있습니다. CollegeSection 검색 결과에서 성균관대학교를 제외하는 방향으로 해결했습니다. - 이메일 인증 (변경 가능) 새 이메일 인증 후 현재 이메일 칸이 업데이트됩니다. - 비밀번호 변경 현재 비밀번호를 입력하고, 새 비밀번호와 확인란이 일치하면 저장 버튼이 활성화됩니다. 현재 비밀번호와 동일한 값으로는 변경할 수 없습니다. ### Additional context - 이메일 알림 아직 백엔드에서 구현되지 않아서 주석 처리했습니다. <img width="1290" height="232" alt="image" src="https://github.com/user-attachments/assets/e90e986e-9208-4753-b75d-5566dca427df" /> - 아직까지 카카오 연동 기능만 구현되어 있습니다. 네이버, 구글, 깃허브 연동 기능은 백엔드 작업 이후 구현 예정입니다. - 로컬 백엔드 환경에서 직접 테스트했습니다. PR 메시지 위주로 검토 부탁드립니다. <!-- e.g. is there anything you'd like reviewers to focus on? --> closes TAS-2731 --- ### Before submitting the PR, please make sure you do the following - [ ] Read the [Contributing Guidelines](https://github.com/skkuding/next/blob/main/CONTRIBUTING.md) - [ ] Read the [Contributing Guidelines](https://github.com/skkuding/next/blob/main/CONTRIBUTING.md#pr-and-branch) and follow the [Commit Convention](https://github.com/skkuding/next/blob/main/CONTRIBUTING.md#commit-convention) - [ ] Provide a description in this PR that addresses **what** the PR is solving, or reference the issue that it solves (e.g. `fixes #123`). - [ ] Ideally, include relevant tests that fail without this PR but pass with it.
|
✅ Syncing Preview App Succeeded Application: |
3 participants
Description
아이디 중복 확인, 이메일 인증, 닉네임 자동 생성, 직업 드롭다운, 대학교 검색 기능을 구현했습니다. 닉네임 중복 확인은 백엔드 API가 없어서 아직 구현하지 못했어요.
성균관대학교 선택 시 이메일 도메인이 @skku.edu로 고정되고 학과 검색과 학번 입력 필드가 추가됩니다. 그 외 대학교는 자유 이메일 입력이고 학과, 학번 항목은 표시되지 않습니다.
Additional context
closes TAS-2691
Before submitting the PR, please make sure you do the following
fixes #123).