Conversation
|
Warning Rate limit exceeded
⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ✨ 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 |
7c1303b to
2d3e7a6
Compare
There was a problem hiding this comment.
DetailRecordV2ResponseDTO 따로 안없애고 그냥 두 개로 두는 걸까요?
There was a problem hiding this comment.
각자 쓰이는 곳이 있어서 이후에 리팩토링하면서 정리할게요
There was a problem hiding this comment.
subemotion을 서버에서 내려주는 이유가 app 단에서 데이터 안갖고 받은 데이터 그대로 뿌릴려고 그랬던 거 같은데, << 이게 맞다면 SubEmotion이 어떤 String으로 구성되면 안될 것 같습니다. 정책을 명확하게 기억하는 상태가 아니라 이거 AOS쪽에 물어보고 후속 작업하시는 거 어떠실까요?
There was a problem hiding this comment.
SubEmotion이 이것저것 하다 남은 레거시 코드네요. 실제로는 사용중이지 않아서 삭제하겠습니다
| case .joy: return BKImage.Graphics.joyCircle | ||
| case .sadness: return BKImage.Graphics.sadCircle | ||
| case .insight: return BKImage.Graphics.insightCircle | ||
| case .other: return BKImage.Graphics.Note.default |
There was a problem hiding this comment.
Circle 이미지를 따로 사용해야 디자인대로 나와서 이 부분은 제가 추후에 수정할게요
There was a problem hiding this comment.
이거 일단은 재사용하긴 했는데 그냥 clip해서 해결했는데 나중에 수정하시면 그것도 지워야 할듯요
There was a problem hiding this comment.
넵 요거는 아마 디자인 QA하면서 나올 것 같아서 그 때 제가 처리하겠습니다
There was a problem hiding this comment.
이거 optional 부분에 추가해둔 toPageString 으로 통일하면 더 좋을 것 같네요
|
아 토끼 리뷰 왜없나했네.. Please wait 16 minutes and 5 seconds before requesting another review. ㅋㅋ |
|
@doyeonk429 아마 3개 pr 한번에 날려서 ratelimit 걸린듯요 ㅋㅋ.. |
ad98006 to
e7bd381
Compare
🔗 관련 이슈
📘 작업 유형
📙 작업 내역
🧪 테스트 내역
🎨 스크린샷 또는 시연 영상 (선택)
✅ PR 체크리스트
💬 추가 설명 or 리뷰 포인트 (선택)