Feat/#31 - Location Repository 구현 및 연결 작업 #35
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
what? - mainTabScene에 caseId 파라미터 추가 why? - 선택된 case Id를 mainTabView에 전달해야하는데... 만족스럽지않다.
what? - Location 데이터 처리를 위한 fetch, delete, create 메서드 구현 - fetch는 복수, delete는 단수, create는 복수로 구현함. - 목데이터 초기 로딩하는 함수를 레포지토리에서 구현 함.(임시) why? - 기본 메서드만 구현, 필요시 추가하기. - 삭제의 경우에는 여러개를 삭제 할 경우가 현재 플로우에선 없음. case삭제 하면 어짜피 연쇄삭제되니. 동시성 에러 만땅. 노란경고 주의
…dd caseId parameter what? - 탭뷰에서 caseId를 추가로 받아, makeMainTabView에서 mapStore에 caseId값을 넘겨주도록 함. - Redux 패턴을 준수하여 MainTabView, MapView의 store를 생성하여 주입시킴 why? - 어느 시점에 데이터 fetch를 하느냐. 탭뷰에서 하는 방법도 있지만 맵뷰에서 했음.(컨테이너를 주로 다루는 곳이 맵이기때문에 했는데 아직 모르겠다.) - Feature에 의존성을 넣는 방식으로 이해하고 있는데, 초기 생성값이 아닌
what? - 맵뷰에 쓸 Location 데이터 모델 정의 - 레포지토리 주입 - Storre 구현, onAppear - fetch - 완료상태 - none. 성공 시locations에 담기!
what? - 주요 함수 fetch, create, delete의 동작을 확인하는 테스트 코드 작성 why? - 잘돌아가는지.. 무사 통과 XCTest 타겟 설정이 잘못되었는지 모듈을 못찾아서 같은 파일에다가 마크주석으로 구분해서 올립니다.
what? - 진짜. 테스트를 위한.. 더미 로더 서비스 - 목데이터 디코딩 모델 포함 why? - 기존 JsonLoader 모듈로 나온 결괏값이 Date 포멧에 안맞아서 그에 따른 포메팅 코드를 추가했습니다. - 목데이터가 기존 양식과 달라 양식이 맞추면 그에 따른 포메팅 모듈로 갈아끼우는 방향으로..
delightPIP
approved these changes
Nov 4, 2025
SUSA24-iOS/SUSA24-iOS/Sources/Presentation/MainTabScene/MainTabView.swift
Show resolved
Hide resolved
mini-min
approved these changes
Nov 4, 2025
SUSA24-iOS/SUSA24-iOS/Sources/Presentation/MapScene/Model/Location.swift
Show resolved
Hide resolved
MuchanKim
changed the title
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
📝 Summary
Location Repository를 구현하였습니다. 연결 작업도 진행했습니다.
🔨 What
👀 Review Notes
Redux + 팩토리 패턴에서 뷰간 발생하는 데이터를 전달하는 방법....
목 날짜 형태가 이상하대서 다시 뽑았는데 링크에서 받아서 리소스에 넣어서 돌려주십쇼!!
임시로 만든 더미 로더는 기존 더미 주간 셀 데이터에서 시간 포멧을 변경한 것입니다.
응애으앵응애