Skip to content

fix(afternote): 수신자 인증 완료 화면 진행 바 미표시 → 100% 채움 (closes 412)#413

Open
1hyok wants to merge 2 commits into
developfrom
feat/412
Open

fix(afternote): 수신자 인증 완료 화면 진행 바 미표시 → 100% 채움 (closes 412)#413
1hyok wants to merge 2 commits into
developfrom
feat/412

Conversation

@1hyok

@1hyok 1hyok commented Jun 21, 2026

Copy link
Copy Markdown
Contributor

📌𝘐𝘴𝘴𝘶𝘦𝘴

Closes #412 — fix(afternote): 수신자 인증 완료 화면 진행 바 미표시 → 100% 채움

📎𝘞𝘰𝘳𝘬 𝘋𝘦𝘴𝘤𝘳𝘪𝘱𝘵𝘪𝘰𝘯

  • 완료 화면(DeliveryVerificationCompleteScreen)이 currentStep=null 로 진행 인디케이터를 숨기던 것을 4/4(100%) 꽉 찬 바 표시로 전환 (704c2a7)
  • ReceiverVerifyStep.COMPLETE=4 추가 + RECEIVER_VERIFY_TOTAL_STEPS 3→4 — 본인확인 25% · 마스터키 50% · 서류 75% · 완료 100% (본인확인은 Intro·Email 두 화면이 공유해 둘 다 25%)
  • 접근성 라벨 receiver_verify_step_description "3단계 중"→"4단계 중" 정합

📷𝘚𝘤𝘳𝘦𝘦𝘯𝘴𝘩𝘰𝘵

에뮬레이터 E2E 실측 — 마스터키 50% · 서류 75% · 완료 100% 진행 바 확인 (완료 화면이 숨김 → 꽉 참).

💬𝘛𝘰 𝘙𝘦𝘷𝘪𝘦𝘸𝘦𝘳𝘴

  • 기존 완료 화면 주석은 "design 9 시안에 진행 인디케이터 없으므로 미표시"였으나, 완료에서 진행을 끝까지 채워 보여주는 것으로 의도 재정리(완료를 4번째 단계로 편입).
  • ⚠️ receiver_verify_step_description 의 총 단계 수가 string 에 하드코딩("4단계 중")이라 RECEIVER_VERIFY_TOTAL_STEPS 와 별도로 관리됨 — 단계 수 변경 시 동반 수정 필요. 근본적으론 %2$d 로 인자화하는 게 낫지만 호출부 5곳 변경이라 이번 범위에선 보류.
  • expiresIn 선제 reissue(feat(core-network): expiresIn 기반 액세스 토큰 선제 reissue (closes 408) #411)와 무관한 독립 변경 (develop 기반).
  • 빌드 검증: ./gradlew :feature:afternote:presentation:compileDebugKotlin :feature:afternote:presentation:ktlintMainSourceSetCheck BUILD SUCCESSFUL

- `ReceiverVerifyStep`에 `COMPLETE`(4단계) 상수를 추가하고 `RECEIVER_VERIFY_TOTAL_STEPS`를 3에서 4로 변경
- `DeliveryVerificationCompleteScreen`에서 진행 인디케이터를 숨기는 대신 100%가 채워진 상태로 표시하도록 수정
- 단계 수 변경에 따라 `receiver_verify_step_description` 문자열 리소스 업데이트
- 수신자 인증 단계 구성(본인 확인 단계의 화면 공유 및 완료 단계 표시 방식 등)에 대한 상세 주석 추가
@1hyok 1hyok added the bug Something isn't working label Jun 21, 2026
@github-actions

Copy link
Copy Markdown

Screenshot test fail — baseline 과 CI rendered PNG 차이.

📦 Artifact: https://github.com/Afternote/Afternote-FE/actions/runs/27892522984 (페이지 하단 Artifacts → screenshot-test-results)

🔍 확인 위치 (artifact zip 내부):

  • diff PNG: preview/debug/diffs/...
  • rendered PNG: preview/debug/rendered/... (CI 환경에서 새로 그린 결과)
  • baseline PNG: repo <module>/src/screenshotTestDebug/reference/... (:core:ui · :app · :feature:onboarding:presentation · :feature:afternote:presentation)

📝 조치 가이드:

  • 의도된 시각 변경 → docker build -t afternote-screenshot:latest -f Dockerfile.screenshot .docker run --rm -v "$PWD":/workspace -w /workspace afternote-screenshot:latest ./gradlew :<module>:updateScreenshotTest 로 baseline 갱신 + commit/push
  • diff 가 anti-aliasing/font 차이로 보이면 docker 환경 변경 여부 (Dockerfile.screenshot) 점검

CI ↔ 로컬 환경 차이는 본 워크플로의 container 단계가 root fix 한다 (README 의 "Compose Preview Screenshot Testing (docker baseline)" 섹션 참조).

@1hyok

1hyok commented Jun 21, 2026

Copy link
Copy Markdown
Contributor Author

CI artifact screenshot-test-resultspreview/debug/rendered/(CI 환경에서 새로 그린 결과)를 :feature:afternote:presentation reference baseline 2개에 이식했습니다:

  • DeliveryVerificationCompleteScreen — 진행 바 숨김 → 4/4(100%)
  • IdentityVerificationIntroScreen — 진행률 33%(1/3) → 25%(1/4)

둘 다 RECEIVER_VERIFY_TOTAL_STEPS 3→4 + 완료 단계 추가에 따른 의도된 시각 변경입니다. 가이드의 docker updateScreenshotTest 와 동일 결과(동일 CI 렌더)이며, 현재 staged 상태 — owner 커밋·push 후 CI 재검증 시 통과합니다.

🤖 Addressed by Claude Code

- `ReceiverVerifyStep`에 `COMPLETE`(4단계) 상수를 추가하고 `RECEIVER_VERIFY_TOTAL_STEPS`를 3에서 4로 변경
- `DeliveryVerificationCompleteScreen`에서 진행 인디케이터를 숨기는 대신 100%가 채워진 상태로 표시하도록 수정
- 단계 수 변경에 따라 `receiver_verify_step_description` 문자열 리소스 업데이트
- 수신자 인증 단계 구성(본인 확인 단계의 화면 공유 및 완료 단계 표시 방식 등)에 대한 상세 주석 추가
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fix(afternote): 수신자 인증 완료 화면 진행 바 미표시 → 100% 채움

1 participant