[카탈로그] 아이템의 종류나 색상을 수집 상태를 기록할 수 있는 기능 추가 #151
Merged
Conversation
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
- VariantCollectionEntity 추가 (variantId, itemName 저장) - VariantsStorage 프로토콜 정의 (fetch, add, remove, removeAll) - CoreDataVariantsStorage 구현 (RxSwift 기반) - CoreDataStorage에 자동 병합 설정 추가 (automaticallyMergesChangesFromParent) - CoreDataItemsStorage에 viewContext 즉시 저장 로직 추가 (데이터 손실 방지)
- WardrobeVariat.toVariat()에서 하드코딩된 "1_0_0" 대신 filename과 internalId를 조합한 고유 ID 생성
- 형식: "{filename}_{internalId}_0"으로 변경하여 각 variant마다 고유성 보장
- variationsWithColor 필터 통과를 위해 "_0" 접미사 유지
- ItemDetailReactor에 variant collection actions/mutations/state 추가 - fetchCollectedVariants, toggleVariantCollection 액션 - collectedVariantIds 상태 관리 - CatalogCellReactor에 아이템 수집 해제 시 variants 일괄 제거 로직 추가 - Items 싱글턴에 variant 관리 기능 추가 - collectedVariants BehaviorRelay 추가 - updateVariant, getCollectedVariants 메서드 추가 - CoreDataVariantsStorage 초기화 및 로드 - 아이템 체크 해제 시 해당 아이템의 모든 variants 자동 제거 - Variant 체크 시 아이템도 자동 수집 처리
- VariantCell에 체크박스 UI 추가 - checkButton (checkmark.seal 아이콘) - showCheckbox 플래그로 리폼 가능 아이템 필터링 - checkboxObservable로 체크 이벤트 전달 - ItemVariantsView에 수집 상태 관리 추가 - collectedVariantIds BehaviorRelay로 reactive 업데이트 - isReformable 플래그로 체크박스 표시 제어 - didToggleVariantCollection Observable 추가 - ItemDetailViewController 바인딩 추가 - viewDidAppear에서 수집 상태 새로고침 - reactor와 양방향 바인딩 (collectedVariantIds ↔ UI) - setUpVariant에서 리폼 가능 여부 전달
- sceneDidEnterBackground에서 background task 시작 - 비동기 CoreData 작업 완료를 위해 2초 대기 후 viewContext 저장 - 앱 강제 종료 시 데이터 손실 방지
|
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 You can disable this status message by setting the
✨ Finishing touches🧪 Generate unit tests (beta)
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 |
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.
📕 Issue
fix ISSUE_URL
📙 작업 내역
📘 작업 유형
📋 체크리스트