Skip to content

feat(fe): impl verification logic for whitelist removal#3552

Open
ojongii wants to merge 9 commits into
feat/remove-whitelistfrom
t2691-impl-verification-logic-for-whitelist-removal
Open

feat(fe): impl verification logic for whitelist removal#3552
ojongii wants to merge 9 commits into
feat/remove-whitelistfrom
t2691-impl-verification-logic-for-whitelist-removal

Conversation

@ojongii
Copy link
Copy Markdown
Contributor

@ojongii ojongii commented May 9, 2026

Description

아이디 중복 확인, 이메일 인증, 닉네임 자동 생성, 직업 드롭다운, 대학교 검색 기능을 구현했습니다. 닉네임 중복 확인은 백엔드 API가 없어서 아직 구현하지 못했어요.

성균관대학교 선택 시 이메일 도메인이 @skku.edu로 고정되고 학과 검색과 학번 입력 필드가 추가됩니다. 그 외 대학교는 자유 이메일 입력이고 학과, 학번 항목은 표시되지 않습니다.

Additional context

closes TAS-2691


Before submitting the PR, please make sure you do the following

@ojongii ojongii requested a review from dayeoni May 9, 2026 13:57
@ojongii ojongii self-assigned this May 9, 2026
@ojongii ojongii added ⛳️ team-frontend preview 이 라벨이 붙어있어야 프론트엔드 Preview 환경이 생성됩니다 labels May 9, 2026
Copy link
Copy Markdown
Contributor

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

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 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.

Comment thread apps/frontend/components/auth/SignUpPage/signup.schema.ts Outdated
Comment thread apps/frontend/components/auth/SignUpPage/SignUpPage.tsx
Comment thread apps/frontend/components/auth/SignUpPage/SignUpPage.tsx Outdated
@skkuding-bot
Copy link
Copy Markdown

skkuding-bot Bot commented May 9, 2026

Syncing Preview App Succeeded

Application: frontend
Revision: 090e50eb14d6c5a1f9a2d813a976f774df159456
Health Status: Healthy

Open Preview | View in Argo CD

@skkuding-bot
Copy link
Copy Markdown

skkuding-bot Bot commented May 9, 2026

Syncing Preview App Succeeded

Application: frontend
Revision: adba6cd149efc7128b0e9379ca1e8566121ea2cc
Health Status: Healthy

Open Preview | View in Argo CD

@skkuding-bot
Copy link
Copy Markdown

skkuding-bot Bot commented May 11, 2026

Syncing Preview App Succeeded

Application: frontend
Revision: f6a03cb7e368eefcb254e24e629d3a0879fd6870
Health Status: Healthy

Open Preview | View in Argo CD

Comment thread apps/frontend/components/auth/SignUpPage/SignUpPage.tsx Outdated
}
}

const generateNickname = () => {
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.

기획안2 보시면 우아한 형제들에서 만든 한글 닉네임을 활용하기로 한걸 확인하실 수 있습니다! 그래서 해당 기획안보시고, 활용하시면 좋을거 같아요!
(링크: https://www.notion.so/skkuding/02-2adef9cff54580d2b143e1c1eae030e5)
*별명추천받는거 가능할까요?에 있는 댓글 참고하시면 됩니다

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

해당 라이브러리 확인해봤는데, 다소 부적절한(?) 단어가 포함되어 있어서 직접 리스트를 만들거나 다른 닉네임을 알아봐서 활용하는 편이 나을 것 같은데.. 혹시 어떻게 생각하시나요?

Copy link
Copy Markdown
Contributor

@dayeoni dayeoni May 22, 2026

Choose a reason for hiding this comment

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

오호..다시 봤는데, '다른 닉네임' 링크 조은데요???!!

Comment thread apps/frontend/components/auth/SignUpPage/SignUpPage.tsx Outdated
<div className="relative flex-1">
<input
type="text"
placeholder="신나는 청사과"
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.

기본 닉네임은 제가 임의로 작성했던거라, 우아한 형제들이 제공하는 기능을 활용하면 닉네임 placeholder를 그 중 하나로 변경하면 될거 같습니다!

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

위 댓글 참고 부탁드립니다!

@dayeoni
Copy link
Copy Markdown
Contributor

dayeoni commented May 13, 2026

[화면 기반 수정사항입니다!]

  • 아이디에 값 입력 후 중복 확인버튼을 누르면, '아이디 중복 확인을 해주세요'가 1초 나온 다음, '사용 가능한 아이디입니다'가 나옵니다!
  • 다른 입력란은 입력->검증 완료 후 border가 사라지는데, 아이디는 중복 확인 이후에도 border가 유지됩니다. 사라지도록 수정부탁드립니다!

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"
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.

화면상에서 '인증하기' 텍스트 크기가 잘 적용되지 않고 있는거 같습니다. '재발송'에 비해 작아보여서 확인부탁드립니다!

Image

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

수정했습니다!

@skkuding-bot
Copy link
Copy Markdown

skkuding-bot Bot commented May 13, 2026

Syncing Preview App Failed

Application: frontend
Revision: t2691-impl-verification-logic-for-whitelist-removal
Health Status: Healthy

Open Preview | View in Argo CD

@nhjbest22
Copy link
Copy Markdown
Contributor

main이 아니라 feat/remove-whitelist로 올려주세요!

…1-impl-verification-logic-for-whitelist-removal
@ojongii ojongii changed the base branch from main to feat/remove-whitelist May 20, 2026 06:08
@skkuding-bot
Copy link
Copy Markdown

skkuding-bot Bot commented May 20, 2026

Syncing Preview App Succeeded

Application: frontend
Revision: a8880dbfb425fffcece6489b70e94c4ec9f2b7bd
Health Status: Healthy

Open Preview | View in Argo CD

@skkuding-bot
Copy link
Copy Markdown

skkuding-bot Bot commented May 23, 2026

Syncing Preview App Succeeded

Application: frontend
Revision: d62f04eff3e092167e9caadc39433b22a0057093
Health Status: Healthy

Open Preview | View in Argo CD

@ojongii
Copy link
Copy Markdown
Contributor Author

ojongii commented May 27, 2026

korean-random-names-generator 닉네임 생성 패키지를 설치하고 placeholder를 변경했습니다.

아래는 지난 회의 노션에 기록된 기획 관련 수정 사항입니다.

  • 일반인도 이메일 인증하기 -> 백엔드 구현이 안 된 상태여서 아직 작업하지 않았습니다.
  • 닉네임은 중복확인 X (고유식별자는 ID, 이메일) -> 중복 확인 버튼 제거하고 관련 프론트 코드 모두 제거했습니다.
  • 생년월일 제거 -> 지난 커밋에서 UI 제거했고, 이번 커밋에서 관련 필드를 스키마에서 제거했습니다.
  • 프로필 생성시, 코드당 캐릭터 안하고, 저번 api랑 프로필 업로드 두개로 진행하기로~ -> 작업 범위와 무관
  • 아이디 기입 시, 공백이나 특수문자(언더바 외)는 사용할 수 없습니다. (→ 중복확인 할 때 아래 빨간색으로 표시) -> 먼저 백엔드에서 언더바가 허용되도록 바뀌어야 할 것 같아서 아직 작업하지 않았습니다!
  • (선택) 마케팅 활용 동의 및 광고 수신 동의 제거 -> 체크박스 UI 제거했습니다!

@skkuding-bot
Copy link
Copy Markdown

skkuding-bot Bot commented May 27, 2026

Syncing Preview App Failed

Application: frontend
Revision: 217299cd9e122f339707657a82ded965cfe450b6
Health Status: Degraded

Open Preview | View in Argo CD

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

Labels

preview 이 라벨이 붙어있어야 프론트엔드 Preview 환경이 생성됩니다 ⛳️ team-frontend

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants