[AN/USER] feat: 학교 상세 페이지 구현 (#697) #722
Conversation
# Conflicts: # android/festago/data/src/main/java/com/festago/festago/data/di/singletonscope/RepositoryModule.kt # android/festago/presentation/src/main/java/com/festago/festago/presentation/ui/home/festivallist/FestivalListFragment.kt # android/festago/presentation/src/main/res/drawable/ic_bookmark_normal.xml
| import com.festago.festago.domain.repository.SchoolRepository | ||
| import javax.inject.Inject | ||
|
|
||
| class SchoolDefaultRepository @Inject constructor( |
There was a problem hiding this comment.
현재 Festival과 Artist는 Fake객체를 반환하는 리포지터리 네이밍은 다음과 같이 구분되어있는데요!
두가지 방법이 있어서 어떤 방식으로 통일할지 생각해보면 좋을 것같습니다!
Fake**Repository
FakeFestivalRepository : FestivalRepository
FestivalDefaultRepository : FestivalRepositoryThere was a problem hiding this comment.
Festival, Artist와 동일하게 FakeSchoolRepository로 변경했습니다. API 연동할 때 SchoolDefaultRepository 추가할게요.
There was a problem hiding this comment.
Fake**Repository
Default**Repository
로 통일하는게 어떤가요?
FakeFestivalRepository : FestivalRepository
DefaultFestivalRepository : FestivalRepository본래 Default 보다 인터페이스 네이밍이 더 잘보이게 하려고 앞에 배치한 것 같은데 권장이나 어순을 고려해서 수식어를 앞에 배치하는게 좋다고 생각합니다:)
| android:padding="8dp"> | ||
|
|
||
| <ImageView | ||
| android:id="@+id/iv_social_media" |
| import com.festago.festago.presentation.R | ||
| import com.festago.festago.presentation.databinding.ViewSocialMediaBinding | ||
|
|
||
| class SocialMediaView @JvmOverloads constructor( |
There was a problem hiding this comment.
이 클래스는 아티스트 화면에도 있는데 화면마다 있어도 괜찮겠죠??
There was a problem hiding this comment.
common 뷰로 두고 같은 뷰 재활용하는 것도 나쁘지 않다고 생각하는데 수정할까요??
There was a problem hiding this comment.
SocialMediaView 제거하고 아티스트 화면에 있는 뷰 사용하도록 변경했어요
|
|
||
| private fun handleSuccess(uiState: SchoolDetailUiState.Success) { | ||
| binding.successUiState = uiState | ||
| binding.ivSchoolBackground.setColorFilter(Color.parseColor("#66000000")) |
There was a problem hiding this comment.
@re4rk 백그라운드 이미지 40% 어둡게 처리는 이미 적용되어 있습니다! Fake로 넣어둔 구글 이미지가 투명 배경이라서 티가 안나는 거에요..ㅎㅎ
| val logoUrl: String, | ||
| val backgroundUrl: String, |
There was a problem hiding this comment.
API 연동이랑 변경된 명세 반영은 따로 이슈 파서 작업하겠습니다!
📌 관련 이슈
✨ PR 세부 내용