-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[feat] [#69] [#35] 관심 학교/축제 를 저장하는 로컬 데이터 베이스 환경 구축, 홈 화면 Api 연결 #88
Conversation
core/data/src/main/kotlin/com/unifest/android/core/data/repository/BoothRepository.kt
Outdated
Show resolved
Hide resolved
core/data/src/main/kotlin/com/unifest/android/core/data/repository/FestivalRepository.kt
Outdated
Show resolved
Hide resolved
core/network/src/main/kotlin/com/unifest/android/core/network/response/FestivalResponse.kt
Outdated
Show resolved
Hide resolved
feature/home/src/main/kotlin/com/unifest/android/feature/home/HomeScreen.kt
Outdated
Show resolved
Hide resolved
feature/home/src/main/kotlin/com/unifest/android/feature/home/viewmodel/HomeViewModel.kt
Outdated
Show resolved
Hide resolved
feature/menu/src/main/kotlin/com/unifest/android/feature/menu/viewmodel/MenuViewModel.kt
Outdated
Show resolved
Hide resolved
feature/menu/src/main/kotlin/com/unifest/android/feature/menu/viewmodel/MenuViewModel.kt
Outdated
Show resolved
Hide resolved
core/database/src/main/kotlin/com/unifest/android/core/database/StarListConverter.kt
Outdated
Show resolved
Hide resolved
feature/home/src/main/kotlin/com/unifest/android/feature/home/HomeScreen.kt
Outdated
Show resolved
Hide resolved
feature/home/src/main/kotlin/com/unifest/android/feature/home/HomeScreen.kt
Show resolved
Hide resolved
생각을 해봤는데 이미 관심축제로 추가되어있는 경우엔 뷰에서 해당 축제를 다시 추가하지 못하도록 조치들을 취하고 있습니다. (추가 버튼이 아닌, 체크 표시로 바뀐다던지, 관심 축제로 추가하는 버튼이 안보인다던지) 그래서 그런 부분에 대해선 뷰모델에서 room 에 데이터를 추가하는 함수에서 예외처리를 할 필요는 없어보여요. onConflictStratey 로 Replace 로 이미 설정해놨다고 리뷰엔 언급을 했는데, 사실 replace 되는것보단 아예 무시되는게 더 맞다고 생각이드네요. 무튼 요악하자면 중복으로 또 추가할 상황 자체가 안만들어질것같다는 말이었습니다. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! 수고하셨습니다.
Todo:
BottomSheet와 홈 화면에서의 관심 축제 추가 기능은 각각 다른 데이터 구조를 사용합니다. 이 두 기능을 위해 통합된 엔티티를 활용하되, 각 기능에서 필요한 데이터만을 저장하도록 하였습니다. 추가적으로, 로컬 데이터베이스에 저장할 때는 이미 같은 festivalId를 가진 데이터가 존재하는지 확인하고, 존재하지 않는 경우에만 새로운 데이터를 저장합니다.