Repository navigation
[Feat] [운세] Fortune 홈 후속 상세 및 사주풀이 3종 구현 (#79) - #94
Merged
Merged
Conversation
- AuthSession 모듈 의존성을 ComposableArchitecture에서 경량 Dependencies로 교체 - DesignSystem bell 아이콘 svg 파일명 대소문자 매핑 불일치 수정
- 사주 만세력 기둥 천간/지지 및 오행 컬러를 표시하는 DSSajuPillarCell 구현 - DSSajuElement 오행(목·화·토·금·수) 테마 및 음양 표기 지원 - Figma 명세 문서(Docs) 및 Example 플레이그라운드·카탈로그 등록 - DesignSystemTests에 Specification 매핑 단위 테스트 추가
- 카테고리별 상세운 바텀시트(FortuneCategoryDetailView) UI 및 레이아웃 구현 - 행운 아이템(클로버), 주의 아이템(경고), AI 토닥이 플로팅 버튼 3D 에셋 추가
- CompatibilityFeature 내 정보 변경 액션을 상위 FortuneFeature 델리게이트로 연결 (#83) - MainTabFeature에서 myPageRequested 수신 시 마이페이지 탭으로 전환 처리
- FortuneCategoryDetailFeature todakRequested delegate 상위 라우팅 누락 수정 - CompatibilityFeature 초기화/목록 조회 Effect에 CancelID 및 취소 방어 적용 - DSSajuPillarCell 10천간 12지지 22자 음양(+/-) 동적 계산 로직 추가 - FortuneCategoryDetailView 하드코딩된 RGB 색상을 Color.ds 토큰으로 대체 - DSSajuPillarCell 22자 오행 파싱 및 MainTab/FortuneFeature 델리게이트 단위 테스트 보강
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
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. Comment |
- FortuneReportView reportBody 본문에 categorySection(detail) 추가
- 상대방 등록 API 전송 시 동료 관계 코드를 COLLEAGUE로 수정 - 시간 모름 시 birthTime 값을 UNKNOWN으로 전송하도록 수정 및 불필요한 isTimeUnknown 필드 제거 - 궁합 결과 화면의 사주 구조도를 Grid 레이아웃으로 변경하여 가변 이름 지원 및 컬럼 밀림 방지 - 상대방 선택 휠피커 바텀시트 추가 버튼 복원 및 닫힘 시 커밋 UX 적용
- DSSajuBasicFieldsView 컴포넌트 신규 구현을 통해 성별, 음/양력, 생년월일, 태어난 시간 입력 폼 공통화 - BirthDate 및 BirthDatePolicy를 Core/Model에 구축하여 윤년, 말일 정규화, 미래 날짜 및 만 14세 유효성 검증 단일화 - FortuneFeature(궁합), MyPageFeature(상대방 관리/수정), OnboardingFeature에 공통 폼 마이그레이션 및 레거시 중복 코드 정리 - UnicodeScalar+Korean을 Core/Utils로 공통화하여 한글 이름 검증 단일화 - 폼 상태 전이, 미래 날짜/만 14세 정책, 궁합/상대방 사주 관리 단위 및 통합 테스트 보강
- 궁합 화면에서 내 정보 수정 후 저장 결과를 즉시 반영합니다.\n- CTA 컨테이너의 투명 배경은 유지하고 disabled 버튼은 DesignSystem 기본 표면을 사용합니다.\n- MainTab·Fortune·MyPage 상태 전이 테스트를 보강합니다.
- 프로필 수정 후 궁합 내 사주 재조회 실패를 오류 배너로 전달합니다.\n- 기존 사주 정보는 유지하는 회귀 테스트를 추가합니다.
- App이 궁합 경로 식별자로 편집 요청의 수명을 관리하도록 변경\n- 뒤로가기와 탭 전환 시 대기 중인 편집 표시와 전용 대시보드 로드를 폐기\n- 늦은 응답이 편집 화면을 다시 표시하지 않도록 회귀 테스트 추가
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.
변경 내용
DSSajuPillarCell을 추가하고, 명세·플레이그라운드·스펙 테스트를 등록했습니다.FortuneClientDTO/Live API 연동을 구현했습니다.MyPageEditView를 제시합니다.profileUpdateddelegate를 통해 현재 Compatibility path의mySajuRefreshRequested로 전달합니다.설계 결정
DSSajuPillarCell은 MyPage와 Fortune 사이의 Feature 구현체 의존성을 만들지 않도록 DesignSystem 공용 컴포넌트로 분리했습니다.MainTab)에 두고, Fortune과 MyPage는 delegate 및 Interface 계약으로만 연결했습니다.테스트
tuist generate --no-openxcodebuild build-for-testing ... -scheme FortuneFeaturexcodebuild test-without-building ... -scheme FortuneFeature -only-testing:FortuneFeatureTestsxcodebuild build-for-testing ... -scheme Todakunxcodebuild test-without-building ... -scheme Todakun -only-testing:TodakunTests/MainTabFeatureTestsmise exec -- swiftlint lint --strict --config .swiftlint.yml./scripts/sync-and-validate.shMainTabFeatureTests,FortuneFeatureTests,FortuneFollowupFeatureTests,MyPageFeatureTests의 관련 상태 전이 테스트 보강PR Preflight Review
gpt-5.6-solhigh 읽기 전용 리뷰myInfoEditRequested → presentEdit → profileUpdated → mySajuRefreshRequested상태 전이가 연결돼 있습니다.제외 범위 및 후속 작업
연결된 이슈
체크리스트