[PC-1298] PR 피드백 수정 #167
Merged
[PC-1298] PR 피드백 수정 #167
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
|
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 |
tgyuuAn
approved these changes
Oct 19, 2025
| Spacer(modifier = Modifier.height(16.dp)) | ||
|
|
||
| Row(modifier = Modifier.fillMaxSize()) { | ||
| ProductList( |
There was a problem hiding this comment.
이 부분, UI레벨에서 말고 처음부터 StoreState로 매핑해줄 때 해주는 것이 더 깔끔해보여요!
tgyuuAn
approved these changes
Oct 20, 2025
Comment on lines
9
to
13
| data class StoreState( | ||
| val products: List<PieceProduct> = emptyList(), | ||
| val puzzleCount : Int = 20, | ||
| val promotionProductList: ImmutableList<PromotionProduct> = persistentListOf(), | ||
| val normalProductList: ImmutableList<NormalProduct> = persistentListOf() | ||
| ) : UiState |
kkh725
added a commit
that referenced
this pull request
Oct 20, 2025
* [PC-958] PlayConsole에서 상품 목록을 가져오는 것 구현 * [PC-958] temp * [PC-958] 인앱 상품을 가지고 오는 로직 개선 * [PC-958] 인앱 상품을 가지고 오는 로직 개선 * [PC-958] Domain, Data를 타던 Billing 로직을 core:billing으로 별도로 분리 * [PC-958] 결제 가능한 상품을 받아올 수 있는 API 연결 * [PC-958] 서버로부터 가용 제품 내려서 Console로 쿼리하는 파이프라인 형성 * [PC-958] 구매 검증 로직 추가 * temp * [PC-958] 가용상품 API Response 변경 대응 * [PC-958] Http Header 로깅 추가 * [PC-958] "/api/cash-products" api의 헤더에 토큰 누락되어 요청되던 현상 해결 * [PC-958] Coil-svg 라이브러리 추가 * [PC-958] PromotionProduct 관련 로직 수정 - PromotionProduct 모델 수정 - PromotionProductResponse 모델 수정 - PreviewPromotionCard UI 삭제 이후 PieceImage로 대체 * [PC-1289] compose-material3-adaptive 의존성 추가 * [PC-1289] StoreScreen 기기별 UI 대응(compact, expand). medium 사이즈는 추후 대응 * [PC-958] StoreScreen 진입 시 consume 되지 않은 상품 처리 구현 * [PC-958] App의 onResume 시마다 consume되지 않은 상품 검증 구현 * [PC-958] 로그인 한 유저만 검증 api 처리하도록 수정 * [PC-958] StoreViewModel의 billingHelper의 접근제어자 internal -> private으로 수정 * [PC-1298] PR 피드백 수정 (#167) * [PC-1298] app/debug 불필요한 파일들 제거 * [PC-1298] compose.material3.adaptive 의존성 store 에만 추가하는 방식으로 수정 * [PC-1298] consumeProductList 의미없는 주석 제거 * [PC-1298] 주석 제거 * [PC-1298] AdaptiveLayout -> feature:store 에서만 사용하게 수정 * [PC-1298] BillingHelperImpl.kt 에서만 사용하는 toPieceProduct 메서드 수정 * [PC-1298] feature:store 모듈에서 billingclient 의존성 제거 * [PC-1298] 주석 제거 * [PC-1298] kotlin.collection.immutable 의존성 추가 및 list->immutableList로 변환 * [PC-1298] 서버에서 consume 작업 및 unConsumed 상품 처리를 담당 -> 클라쪽 처리 로직 삭제. * [PC-1298] StoreState의 products -> promotion, normal 으로 분리, puzzleCount 상태 추가 * [PC-1298] 뷰모델에서 액티비티 제거 후 sideEffect로 분리 * [PC-958] TopBar 병합 중 누락된 이슈 대응 --------- Co-authored-by: kkh <[email protected]> Co-authored-by: Kim Kuen Hyung <[email protected]>
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.
1. ⭐️ 변경된 내용
2. 🖼️ 스크린샷(선택)
3. 💡 알게된 부분
4. 📌 이 부분은 꼭 봐주세요!