Conversation
…tification-poke # Conflicts: # feature/task-certification/src/main/java/com/twix/task_certification/detail/contract/TaskCertificationDetailUiState.kt # feature/task-certification/src/main/java/com/twix/task_certification/navigation/TaskCertificationGraph.kt
- `ic_photolog_action_poke.xml` 벡터 드로어블 리소스 추가
…cation-poke # Conflicts: # feature/task-certification/src/main/java/com/twix/task_certification/detail/contract/TaskCertificationDetailUiState.kt
…cation-poke # Conflicts: # feature/stats/detail/src/main/java/com/twix/stats/detail/StatsDetailViewModel.kt
|
@coderabbitai review |
✅ Actions performedReview triggered.
|
📝 WalkthroughWalkthroughBackgroundCard 컴포저블 시그니처가 변경되어 Estimated code review effort🎯 5 (Critical) | ⏱️ ~120 minutes 🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ 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 |
이슈 번호
#112
작업내용
결과물
리뷰어에게 추가로 요구하는 사항 (선택)
리뷰 참고
리뷰 완료되지 않은 작업 내용이 필요해서 우선 110번 브랜치에서 작업한 내용을 받아서 작업한 관계로 110번 브랜치와 커밋이 섞여있어 !이 PR들 먼저 리뷰하고 리뷰 하면 좋을 것 같아서 PR만 만들어놓고 Request는 두 개먼저 머지되고 요청할게 !
문제 상황 1
현재 ReactionEffect가 종료되었을 때 콜백으로 ViewModel에 MyReactionEffected를 전달해 상태를 업데이트해서
한 번만 리액션이 렌더링 되도록 구현했는데, ReactionEffect 콜백이 호출되기 전에 스와이프하면 상태가 업데이트 되지 않아
계속 ReactionEffect가 렌더링 되는 문제가 존재해
이를 위해 ReactionEffectSpec의 durationRange를 조정해서 사용자가 리액션을 인지하는 최소한의 시간동안 상태를 업데이트 하도록
조정했는데 이 방법도 화면에 진입하자마자 스와이프하면 동일한 문제가 발생해 😓
그래서 리액션을 발생 시키고 LaunchEffect를 통해 상태를 업데이트 하는 방법을 시도해봤는데 이건 아에 리액션 효과가 렌더링 되지 않더라구 ☠️
계속 고민해보다 암만 생각해도 더 나은 방법을 모르겠어서 현수 의견도 들어보고 싶어가지구 우선 PR 올렸어 !!
또한 간헐적으로 리액션 효과 렌덩시 버벅임이 발생해서 이건 계속 해결 방법을 고민중인데 시간이 좀 걸릴 것 같아 🥲
요것두 좋은 생각이 있다면 공유해주면 좋을 것 같아 !
문제 상황 2
MyReactionBadge 컴포넌트에 최대한 디자인과 비슷하게 아이콘을 배치시켜 놨는데 배경과 아이콘이 분리되어있다 보니
디자인 요구사항을 100% 충족하는 상황은 아닌 것 같아
코드로 배치시키는건 한계가 있을거 같아서 따로 아이콘별로 디자이너들한태 요청할지 아니면 그대로 갈지 현수 의견이 궁금해 !
요청 사항
피그마에 아이콘 업데이트 돼서 기존에 사용하던 리액션 아이콘들을 교체했어 !

홈 화면용 따로 도운이가 만들어 놓은 것 같은데 한번 디자이너 칭구들한태 교체해야하는지 확인 부탁해 !