feat: 프로필 실데이터 연동(연락처·로그인이력·보안·알림설정) + 회원가입 전화번호 - #342
Open
BcKmini wants to merge 5 commits into
Open
Conversation
ProfilePage가 하드코딩된 profileData.ts 대신 실제 GET/PATCH /api/v1/auth/me/profile(fowoco/server#171)를 쓰도록 교체 — 저장한 내용이 새로고침해도 유지된다. 로그인 시에도 서버가 이미 보내주던 display_name을 그동안 안 쓰고 이메일 앞부분으로 가짜 이름을 만들던 것을 고쳤다. 회원가입 화면에 전화번호(선택) 입력 필드 추가. 서버에 없는 '이름'(name, displayName과 중복)·선호 언어·시간대 필드는 저장이 안 되는데도 있는 것처럼 보이던 문제라 이번에 편집 항목에서 제외함. Closes #335 Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
ProfilePage에 남아있던 나머지 하드코딩(fowoco/server PR #171에서 새로 추가된 로그인 이력·계정 상태·비밀번호 변경일·알림 수신 설정 API)을 화면에 실제로 연결한다. - 마지막 로그인 시각/기기, 계정 보호 상태, 비밀번호 변경일을 GET /auth/me/profile의 실제 값으로 표시. - "업무 Context와 권한"에서 서버에 대응 개념이 없던 담당 업무 영역·문서 열람 범위 항목은 제거하고, role 기반으로 실제로 검증되는 승인/자료 등록 권한만 표시. - 개인 알림 설정을 GET/PATCH /notifications/preferences에 연결 — 토글하면 실제로 저장되고 새로고침해도 유지된다. 라벨/설명 문구는 서버에 없는 UI 전용 카피라 profileData.ts에 유지. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
…t/335-profile-phone
profile.ts(add/add), profileData.ts, ProfilePage.tsx, ProfilePage.test.tsx 충돌. main 쪽은 WORK_CONTEXT/SECURITY_INFO/알림 enabled를 여전히 정적 데모값으로 갖고 있는 더 이전 상태였고, 이 브랜치가 실제 서버 API(계정상태·비밀번호 변경일·로그인이력·알림 설정)로 이미 대체한 버전이라 이 브랜치 쪽으로 해결. main에만 있던 required(필수 알림 토글 불가) 로직은 이 브랜치 ProfilePage.tsx 안에도 이미 동일하게 남아있어 유실 없음 확인. lint/test(455)/build 재검증 완료.
BcKmini
changed the title
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
연결 작업
변경 이유
프로필 화면의 표시이름/연락처 외 나머지 항목(업무 Context, 보안 정보, 알림 설정, 로그인 이력)이 여전히 정적 데모 데이터였음. server#168·#172·#174가 관련 API를 제공하면서 실데이터로 교체.
변경 내용
ProfileResponse확장 필드 및notification_preferenceAPI 사용WORK_CONTEXT,SECURITY_INFO, 고정enabled알림 목록) 제거검증
실행한 명령과 결과:
main과의 merge conflict(profile.ts, profileData.ts, ProfilePage.tsx, ProfilePage.test.tsx) 해결 후 재검증 완료 — main 쪽 정적 데모 버전보다 이 브랜치의 실데이터 연동 버전이 최신이라 이 브랜치 기준으로 해결함. main에만 있던
required(필수 알림 토글 비활성화) 로직은 이 브랜치 컴포넌트 안에 이미 동일하게 구현되어 있어 유실 없음.영향 범위
개인정보·보안
리뷰 포인트