-
Notifications
You must be signed in to change notification settings - Fork 1
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
[Feature/home_network] home/store, home/mypage 네트워크 작업 #79
Conversation
11a6971
to
62afdf0
Compare
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.
👍🏻 리뷰 확인해주세요.
app/src/main/AndroidManifest.xml
Outdated
@@ -64,7 +64,7 @@ | |||
android:exported="false" /> | |||
<activity | |||
android:name=".feature.home.HomeActivity" | |||
android:exported="false" /> | |||
android:exported="true" /> |
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.
이거 다시 false로 변경
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.
넵!
@@ -54,6 +55,7 @@ object NetModule { | |||
): OkHttpClient = OkHttpClient.Builder() | |||
.addInterceptor(logInterceptor) | |||
.addInterceptor(authInterceptor) | |||
.apply { FlipperInitializer.initOkHttpClient(this) } |
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.
👍🏻
override suspend fun getMyPageInfo(): Result<MyPageResponse> { | ||
return runCatching { retrofitMyPageNetwork.getMyPages() } | ||
} | ||
|
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.
빈줄!
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.
감사합니다 ㅎㅎ
val intent = Intent(context, AlbumListActivity::class.java).apply { | ||
putExtra("albumId", albumItem.id) | ||
} |
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.
이거 AlbumListActivity내의 로직으로 전환하는게 좋을듯 cc. @KwakEuiJin
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.
그럼 일단 �인텐트로 엑티비티 전환처리만 해두겠습니다
|
||
binding.viewpagerStore.adapter = adapter | ||
private fun setupViewPager() { | ||
storeAdapter = StoreAdapter({ albumItem -> |
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.
trailing lambda로 처리해주실래유?
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.
넵!!
storeAdapter?.submitList(storeState.data) | ||
|
||
//최초 데이터 세팅 | ||
binding.tvStoreAlbumPhotoCount.text = storeState.data[0].photoCount.toString() + "/30" |
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.
binding.tvStoreAlbumPhotoCount.text = storeState.data[0].photoCount.toString() + "/30" | |
binding.tvStoreAlbumPhotoCount.text = storeState.data.firstOrNull().photoCount.toString() + "/30" |
+) 앨범 없을때의 로직도 추가해야할듯
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.
아래 onPageChanged 콜백하고 동일 로직인 것 같은데 확인해보실?
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.
확인해보니 맞습니다 수정반영 했습니다!
val text = buildSpannedString { | ||
color(colorOf(R.color.pophory_purple)) { | ||
textAppearance(requireContext(), R.style.TextAppearance_Pophory_HeadLineBold) { | ||
append("포포리 앨범") |
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.
append("포포리 앨범") | |
append(coloredText) |
interface OnPageChangedListener { | ||
fun onPageChanged(albumItem: AlbumItem) | ||
} |
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.
interface OnPageChangedListener { | |
fun onPageChanged(albumItem: AlbumItem) | |
} | |
fun interface OnPageChangedListener { | |
fun onPageChanged(albumItem: AlbumItem) | |
} |
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.
반영하겠습니다!
) | ||
} | ||
} | ||
|
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.
빈줄
1f58ecd
to
96367a7
Compare
이슈 코드
📸 스크린샷
🍀 관련 이슈