Skip to content

fix: 인증하기 QA 반영#199

Merged
jihun32 merged 5 commits intodevelopfrom
fix/#146
Feb 25, 2026
Merged

fix: 인증하기 QA 반영#199
jihun32 merged 5 commits intodevelopfrom
fix/#146

Conversation

@jihun32
Copy link
Contributor

@jihun32 jihun32 commented Feb 25, 2026

@clxxrlove
Copy link
Member

@coderabbitai summary

@coderabbitai
Copy link

coderabbitai bot commented Feb 25, 2026

ℹ️ Recent review info

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between a1fb4af and b04a27b.

📒 Files selected for processing (6)
  • Projects/Feature/GoalDetail/Sources/Detail/GoalDetailReducer+Impl.swift
  • Projects/Feature/ProofPhoto/Sources/ProofPhotoReducer+Impl.swift
  • Projects/Feature/ProofPhoto/Sources/ProofPhotoView.swift
  • Projects/Shared/DesignSystem/Sources/Components/Button/ShadowButton/TXShadowButton.swift
  • Projects/Shared/DesignSystem/Sources/Components/TextField/TXCommentCircle.swift
  • Projects/Shared/DesignSystem/Sources/Resources/Typography/View+Typography.swift

📝 Walkthrough

인증샷 등록 QA 반영 - 상대 완료 & 나 미완료 시 오류 수정

핵심 아키텍처 변경

상태 및 액션 흐름 수정

  • ProofPhotoReducer: completedUploadPhoto 델리게이트 액션 발송 시 editedImageData에 실제 이미지 데이터(imageData)를 전달하도록 변경

    • 편집 모드: 사용자가 선택/촬영한 이미지 데이터 전달
    • 일반 모드: 업로드 후 응답받은 이미지 데이터 전달
  • GoalDetailReducer: 델리게이트 액션 바인딩 패턴 수정으로 editedImageData 올바르게 처리

    • proofPhoto(.delegate(.completedUploadPhoto(myPhotoLog, editedImageData))) 패턴으로 변경
    • state.pendingEditedImageData = editedImageData로 저장하여 포토로그 수정 시 활용

비동기 작업 흐름

  • 포토로그 수정 시: pendingEditedImageData가 존재하면 이미지 업로드, 없으면 기존 URL 유지
  • 이미지 최적화 로직(ImageUploadOptimizer.optimizedJPEGData) 일관성 있게 적용

디자인 시스템 개선

TXCommentCircle 컴포넌트 오버홀

  • 기존 단순 원형 레이아웃에서 통합 기하학 기반 구현으로 변경
  • PositionedCircleShape를 활용한 합성 원형 렌더링 (경계선 + 채우기)
  • 텍스트 및 커서 블링킹 효과 추가
  • 외부 포커스 바인딩 지원으로 키보드 상태 관리 개선

기타 UI 조정

  • ProofPhotoView:

    • topBar 후행 패딩(10) 추가
    • Close 버튼 명시적 프레임(24x24) 및 resizable 적용
    • uploadControls/captureControls 조건부 렌더링 단순화
    • 갤러리/캡처 버튼 명시적 프레임(84x84) 추가
    • floatingCommentOverlay 하단 패딩 증가(26→28)
  • TXShadowButton: shadowTopPadding 기본값 조정(4→6)

타이포그래피 수정

  • View+Typography: .kerning().tracking() 변경으로 더 정확한 문자 간격 제어

테스트 커버리지

  • 변경사항에 따른 새로운 테스트 추가 없음 (기존 테스트로 커버)

사용자 영향

  • 상대 사용자가 완료했을 때 현재 사용자의 인증샷 등록 화면 오류 해결
  • 인증샷 등록 및 수정 흐름에서 이미지 데이터 정상 전달
  • 개선된 UI/UX로 더 나은 인증샷 작성 경험 제공

개요

이 변경사항은 사진 증명(ProofPhoto) 기능의 리듀서 로직을 조정하고, UI 컴포넌트의 레이아웃과 시각적 스타일을 개선합니다. 패턴 매칭 문법 업데이트, 편집된 이미지 데이터 전달 변경, 그리고 다양한 UI 요소의 간격, 프레임, 도형 렌더링을 정밀 조정합니다.

변경사항

Cohort / File(s) 요약
리듀서 로직
Projects/Feature/GoalDetail/.../GoalDetailReducer+Impl.swift, Projects/Feature/ProofPhoto/.../ProofPhotoReducer+Impl.swift
ProofPhoto 대리자 완료 케이스의 패턴 매칭 문법 수정 및 편집된 이미지 데이터 페이로드를 nil에서 실제 imageData로 변경.
ProofPhoto 뷰 컴포넌트
Projects/Feature/ProofPhoto/Sources/ProofPhotoView.swift
상단 바 패딩 추가, 닫기 버튼 크기 조정(24x24), 하단 컨트롤 레이아웃 리팩토링, 갤러리/캡처 버튼 명시적 프레임 설정(84x84), 미리보기 컨테이너 모서리 반경 스타일 변경, 플로팅 오버레이 패딩 조정.
디자인 시스템 컴포넌트
Projects/Shared/DesignSystem/Sources/Components/Button/.../TXShadowButton.swift, Projects/Shared/DesignSystem/Sources/Components/TextField/TXCommentCircle.swift, Projects/Shared/DesignSystem/Sources/Resources/Typography/View+Typography.swift
TXShadowButton의 shadowTopPadding 기본값 4→6 조정, TXCommentCircle을 단순 원형에서 병합된 도형 기반으로 전면 리디자인(새로운 기하학 유틸리티 추가, 텍스트 렌더링 개선), 타이포그래피 kerning을 tracking으로 변경.

예상 코드 리뷰 난이도

🎯 3 (Moderate) | ⏱️ ~25분

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/#146

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@coderabbitai
Copy link

coderabbitai bot commented Feb 25, 2026

@clxxrlove, PR 요약을 생성하겠습니다.

✅ Actions performed

Summary regeneration triggered.

@jihun32 jihun32 merged commit 1635b6f into develop Feb 25, 2026
6 checks passed
@jihun32 jihun32 deleted the fix/#146 branch February 25, 2026 12:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants