Skip to content

feat(fe): make settings page for whitelist removal#3636

Open
ojongii wants to merge 18 commits into
t2691-impl-verification-logic-for-whitelist-removalfrom
t2731-make-settings-page-for-whitelist-removal
Open

feat(fe): make settings page for whitelist removal#3636
ojongii wants to merge 18 commits into
t2691-impl-verification-logic-for-whitelist-removalfrom
t2731-make-settings-page-for-whitelist-removal

Conversation

@ojongii

@ojongii ojongii commented Jul 5, 2026

Copy link
Copy Markdown
Contributor

Description

image image image
  • 닉네임 (수정 가능)
    0자 이상 20자 이하입니다. 닉네임이 없으면 빈 칸으로 표시되고, 실제 API 데이터가 오면 기존 닉네임으로 채워집니다. 닉네임이 기존값과 달라지면 저장 버튼이 활성화됩니다.

  • 직업 (현재 수정 불가)
    화이트리스트 제거 이전에 가입한 기존 회원은 jobType이 null로 저장되어 있어 설정 페이지에서 "미설정"으로 표시됩니다. 이 사용자들이 직접 직업을 설정할 수 있도록 수정이 필요합니다. (논의 필요)

  • 대학교 (수정 가능)
    문제점: 타 대학 ↔ 성균관대로 변경할 경우 @skku.edu 이메일 사용 여부 문제가 있습니다.

성균관대 → 타 대학의 경우, isSKKU가 studentId !== '0000000000'으로 결정되기 때문에 SKKU 학번이 있는 사용자는 설정 페이지에서 대학교 필드 자체가 읽기 전용으로 표시됩니다. 이쪽은 이미 막혀 있습니다.

타 대학 → 성균관대의 경우, 대학교를 성균관대학교로 저장하면 SKKU 학생처럼 동작하는 문제가 있습니다.

해결 방법은 크게 두 가지입니다.

프론트에서 막기 — CollegeSection 검색 결과에서 "성균관대학교"를 제외
백엔드에서 막기 — college가 성균관대일 경우 @skku.edu 이메일인지 검증
백엔드 팀과 논의가 필요할 것 같습니다. 일단 프론트에서 막는 방향으로 적용했습니다.

  • 이메일 인증 (변경 가능)
    새 이메일 인증 후 현재 이메일 칸이 업데이트됩니다.

  • 비밀번호 변경
    현재 비밀번호를 입력하고, 새 비밀번호와 확인란이 일치하면 저장 버튼이 활성화됩니다. 현재 비밀번호와 동일한 값으로는 변경할 수 없습니다.

Additional context

  • 이메일 알림
    아직 백엔드에서 구현되지 않아서 주석 처리했습니다.
image
  • 아직까지 카카오 연동 기능만 구현되어 있습니다. 네이버, 구글, 깃허브 연동 기능은 백엔드 작업 이후 구현 예정입니다.

  • 로컬 백엔드 환경에서 직접 테스트했습니다. PR 메시지 위주로 검토 부탁드립니다.

closes TAS-2731


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

@ojongii ojongii added the preview 이 라벨이 붙어있어야 프론트엔드 Preview 환경이 생성됩니다 label Jul 5, 2026

@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 settings and sign-up pages to introduce a more modular component structure, including new sections for account linking, college/major selection, email verification, and profile photos. Key feedback focuses on fixing state synchronization issues in the college and major search inputs when profile data loads asynchronously, correcting the save button's enablement logic which currently triggers prematurely, and implementing the missing file input handler for profile photo uploads. Additionally, several validation and state-resetting bugs in the sign-up flow need to be addressed, such as requiring student IDs for SKKU students and properly clearing university-related fields when switching jobs.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

Comment thread apps/frontend/app/(client)/(main)/settings/_components/CollegeSection.tsx Outdated
Comment thread apps/frontend/app/(client)/(main)/settings/_components/MajorSection.tsx Outdated
Comment thread apps/frontend/app/(client)/(main)/settings/page.tsx
Comment thread apps/frontend/app/(client)/(main)/settings/_components/CollegeSection.tsx Outdated
Comment thread apps/frontend/app/(client)/(main)/settings/_components/MajorSection.tsx Outdated
Comment thread apps/frontend/components/auth/SignUpPage/SignUpPage.tsx
Comment thread apps/frontend/components/auth/SignUpPage/SignUpPage.tsx
Comment thread apps/frontend/components/auth/SignUpPage/SignUpPage.tsx Outdated
@ojongii
ojongii changed the base branch from feat/remove-whitelist to t2691-impl-verification-logic-for-whitelist-removal July 5, 2026 07:18
@skkuding-bot

skkuding-bot Bot commented Jul 5, 2026

Copy link
Copy Markdown

Syncing Preview App Succeeded

Application: frontend
Revision: 60159b1f55928eb60adb23ca37e079d73b8b4fbb
Health Status: Healthy

Open Preview | View in Argo CD

@ojongii
ojongii force-pushed the t2731-make-settings-page-for-whitelist-removal branch from 60159b1 to 3e62851 Compare July 6, 2026 03:33
@skkuding-bot

skkuding-bot Bot commented Jul 6, 2026

Copy link
Copy Markdown

Syncing Preview App Succeeded

Application: frontend
Revision: 3e628510c46b82dc50ebf922c01592054ce6dda6
Health Status: Healthy

Open Preview | View in Argo CD

@ojongii
ojongii force-pushed the t2691-impl-verification-logic-for-whitelist-removal branch from 59c73ef to f308a97 Compare July 6, 2026 03:37
@skkuding-bot

skkuding-bot Bot commented Jul 6, 2026

Copy link
Copy Markdown

Syncing Preview App Succeeded

Application: frontend
Revision: c31cb4e02fd1ec601398abdfe6b8bfa9ca955a69
Health Status: Healthy

Open Preview | View in Argo CD

@skkuding-bot

skkuding-bot Bot commented Jul 6, 2026

Copy link
Copy Markdown

Syncing Preview App Succeeded

Application: frontend
Revision: d98bc82fc45a6853f41e5f9da2786038de8928d1
Health Status: Healthy

Open Preview | View in Argo CD

@ojongii ojongii self-assigned this Jul 6, 2026
@skkuding-bot

skkuding-bot Bot commented Jul 6, 2026

Copy link
Copy Markdown

Syncing Preview App Succeeded

Application: frontend
Revision: 8dee318173918c768432542422d0f9cdaa9eec9b
Health Status: Healthy

Open Preview | View in Argo CD

@ojongii
ojongii force-pushed the t2731-make-settings-page-for-whitelist-removal branch from b6c7709 to 6eaf0c5 Compare July 6, 2026 05:15
@skkuding-bot

skkuding-bot Bot commented Jul 6, 2026

Copy link
Copy Markdown

Syncing Preview App Succeeded

Application: frontend
Revision: 6eaf0c551fd4cb1b82016ae18affe3729123b502
Health Status: Healthy

Open Preview | View in Argo CD

@skkuding-bot

skkuding-bot Bot commented Jul 13, 2026

Copy link
Copy Markdown

Syncing Preview App Succeeded

Application: frontend
Revision: 92096265b08ee8a7b51d7091561c9346be506043
Health Status: Healthy

Open Preview | View in Argo CD

@skkuding-bot

skkuding-bot Bot commented Jul 13, 2026

Copy link
Copy Markdown

Syncing Preview App Succeeded

Application: frontend
Revision: e57a42af94ab51c2fbedca1106954e75cf0683c2
Health Status: Healthy

Open Preview | View in Argo CD

@skkuding-bot

skkuding-bot Bot commented Jul 14, 2026

Copy link
Copy Markdown

Syncing Preview App Succeeded

Application: frontend
Revision: e9427b3a452682338eb4e2cf27ff3cf6e0a7afb5
Health Status: Healthy

Open Preview | View in Argo CD

@ojongii
ojongii requested a review from dayeoni July 16, 2026 04:25
@ojongii ojongii added this to Codedang Jul 16, 2026
@github-project-automation github-project-automation Bot moved this to Pending ✋ in Codedang Jul 16, 2026
@skkuding-bot

skkuding-bot Bot commented Jul 16, 2026

Copy link
Copy Markdown

Syncing Preview App Succeeded

Application: frontend
Revision: ecd3484045609a19377cb0b1f64ef84847c2cd93
Health Status: Healthy

Open Preview | View in Argo CD

@skkuding-bot

skkuding-bot Bot commented Jul 16, 2026

Copy link
Copy Markdown

Syncing Preview App Succeeded

Application: frontend
Revision: 5a9e4c96dc848a40b3ecf13e3ae52124994a1abb
Health Status: Healthy

Open Preview | View in Argo CD

@skkuding-bot

skkuding-bot Bot commented Jul 16, 2026

Copy link
Copy Markdown

Syncing Preview App Failed

Application: frontend
Revision: dec3a9f1eedde3aa43fd60dd674bce33fe3a397d
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

Status: Pending ✋

Development

Successfully merging this pull request may close these issues.

1 participant