Skip to content

fix: 설정 홈에 등록된 프로필 사진 표시 (#2225) - #2233

Merged
OnlyReadOneBook merged 7 commits into
developfrom
fix/2225-setting-profile-avatar
Oct 6, 2026
Merged

OnlyReadOneBook merged 7 commits into
developfrom
fix/2225-setting-profile-avatar

Conversation

@OnlyReadOneBook

@OnlyReadOneBook OnlyReadOneBook commented Oct 1, 2026 •

Copy link
Copy Markdown
Collaborator

확인할 핵심 파일:

  • SettingContract.kt: 서버 사진 URL을 보관하는 프로필 상태
  • SettingViewModel.kt: 조회 결과를 최종 MVI 상태로 전달
  • SettingScreen.kt: 프로필 상태와 아바타 연결
  • SettingProfile.kt: 사진 표시와 기본 이미지 fallback

등록된 프로필 사진을 설정 첫 화면에도 표시합니다. 사진 URL이 없거나 비어 있거나 이미지 요청이 실패하면 기존 기본 자산을 그리며, 새 URL이나 사진 삭제 결과가 도착하면 이전 사진을 남기지 않습니다. 모든 아바타 상태는 같은 60dp 원형 영역을 사용합니다.

프로필 조회 상태를 전환한 #2231 위에 쌓았습니다. 재진입 시점과 하위 화면 갱신은 #1920 범위로 유지합니다.

검증: 설정 presentation 테스트 188건과 ktlint 통과. 새 회귀 6건은 URL 갱신·삭제·조회 실패와 실제 이미지 픽셀 변경·기본 이미지 복귀를 확인합니다. 이미지 요청은 테스트 전용 로더로 격리했습니다. 기본 이미지·가로 사진·로드 실패 Preview 3개는 CI 표준 환경에서 baseline 생성·검증과 PNG 눈검증을 마쳤습니다.

Fixes #2225

CI Test Plan

{
  "androidTest": {
    "mode": "none",
    "reason": "설정 홈의 프로필 URL 상태 전달과 아바타 표시만 변경합니다. URL 교체·삭제·실패 상태는 ViewModel 회귀 3건과 Robolectric 실제 이미지 전체 픽셀 회귀 3건으로 확인했습니다. Activity·navigation·권한·저장소·기기별 API 경계 변경은 없습니다.",
    "tests": []
  }
}

@github-actions
github-actions Bot requested a review from Sadturtleman October 1, 2026 12:01
@github-actions github-actions Bot added the sibling-conflict 열린 형제 PR 과 같은 파일에서 충돌 — 나중에 머지되는 쪽이 base 를 병합해야 한다. 해소되면 자동으로 떨어진다 label Oct 1, 2026
@github-actions

github-actions Bot commented Oct 1, 2026 •

Copy link
Copy Markdown
Contributor

형제 충돌이 해소됐다 — 지금은 이 PR 과 같은 파일을 서로 다르게 고친 열린 PR 이 없다 (#1750).
다시 생기면 이 코멘트를 현재 상태로 고쳐 쓴다.

@OnlyReadOneBook OnlyReadOneBook added the screenshot-baseline CI 표준 환경에서 PR screenshot baseline 생성·적용 요청 label Oct 1, 2026
@github-actions

github-actions Bot commented Oct 1, 2026

Copy link
Copy Markdown
Contributor

📸 CI 표준 환경에서 screenshot baseline 3개를 생성·검증해 c35314f 로 커밋했습니다.
실행 로그·생성 artifact: https://github.com/Afternote/Afternote-FE/actions/runs/36859924751
이미지 내용은 이 PR의 PNG diff에서 눈으로 최종 확인해 주세요. 필수 검사도 새 SHA에 다시 요청했습니다.

@github-actions github-actions Bot removed the screenshot-baseline CI 표준 환경에서 PR screenshot baseline 생성·적용 요청 label Oct 1, 2026

@Sadturtleman Sadturtleman left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

설정 홈 프로필에 서버 사진 URL을 전달하고 빈 값·로드 실패 시 기본 이미지로 돌아가는 흐름을 확인했습니다. 승인합니다.

@github-actions github-actions Bot removed the sibling-conflict 열린 형제 PR 과 같은 파일에서 충돌 — 나중에 머지되는 쪽이 base 를 병합해야 한다. 해소되면 자동으로 떨어진다 label Oct 4, 2026
@github-actions github-actions Bot added sibling-conflict 열린 형제 PR 과 같은 파일에서 충돌 — 나중에 머지되는 쪽이 base 를 병합해야 한다. 해소되면 자동으로 떨어진다 and removed sibling-conflict 열린 형제 PR 과 같은 파일에서 충돌 — 나중에 머지되는 쪽이 base 를 병합해야 한다. 해소되면 자동으로 떨어진다 labels Oct 5, 2026
@github-actions github-actions Bot added the conflict base 와 충돌 — CI 가 트리거되지 않는다. base 를 병합하면 자동으로 떨어진다 label Oct 6, 2026
@github-actions

github-actions Bot commented Oct 6, 2026

Copy link
Copy Markdown
Contributor

이 PR 은 fix/708-setting-error-retry 와 충돌 상태이고, 그래서 CI 가 돌지 않는다.

pull_request 워크플로는 base 와 head 를 합친 가상 머지 커밋(refs/pull/<N>/merge)에서
실행되는데, 충돌이면 GitHub 이 그 ref 를 만들지 못해 워크플로가 트리거조차 되지 않는다.
체크가 실패하는 것이 아니라 아예 생성되지 않으므로, 새 커밋을 올려도 no checks reported
인 채 최신 커밋이 한 번도 검증되지 않은 상태로 남는다 (#1028).

fix/708-setting-error-retry 를 병합해 충돌을 풀면 검증이 다시 돈다. 해소되면 이 라벨은 자동으로 떨어진다.

@github-actions github-actions Bot removed the conflict base 와 충돌 — CI 가 트리거되지 않는다. base 를 병합하면 자동으로 떨어진다 label Oct 6, 2026
Base automatically changed from fix/708-setting-error-retry to develop October 6, 2026 07:11
@OnlyReadOneBook
OnlyReadOneBook added this pull request to the merge queue Oct 6, 2026
Merged via the queue into develop with commit e30f607 Oct 6, 2026
37 of 38 checks passed
@OnlyReadOneBook
OnlyReadOneBook deleted the fix/2225-setting-profile-avatar branch October 6, 2026 07:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

android-test PR head에서 Android Managed Device 계측 테스트 실행

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fix(setting): 설정 홈 아바타에 등록된 서버 프로필 사진이 표시되지 않는다

2 participants