Skip to content

feat: 기록 수정 감정 선택 UI 개선#277

Merged
clxxrlove merged 4 commits intodevelopfrom
BOOK-508-feature/#272
Feb 15, 2026
Merged

feat: 기록 수정 감정 선택 UI 개선#277
clxxrlove merged 4 commits intodevelopfrom
BOOK-508-feature/#272

Conversation

@clxxrlove
Copy link
Member

🔗 관련 이슈

📘 작업 유형

  • ✨ Feature (기능 추가)
  • 🐞 Bugfix (버그 수정)
  • 🔧 Refactor (코드 리팩토링)
  • ⚙️ Chore (환경 설정)
  • 📝 Docs (문서 작성 및 수정)
  • ✅ Test (기능 테스트)
  • 🎨 style (코드 스타일 수정)

📙 작업 내역

  • 독서 기록 및 수정 플로우 UI 개편
  • 독서 상세 플로우 UI 개편
  • 기타 감정 추가 및 반영 (감정 카드 등)
  • API 버저닝으로 인해 xcconfig 값 변경 및 APIConfig 수정

🧪 테스트 내역

  • 브라우저/기기에서 동작 확인
  • 엣지 케이스 테스트 완료
  • 기존 기능 영향 없음

🎨 스크린샷 또는 시연 영상 (선택)

기능 미리보기
기능 설명

✅ PR 체크리스트

  • 커밋 메시지가 명확합니다
  • PR 제목이 컨벤션에 맞습니다
  • 관련 이슈 번호를 작성했습니다
  • 기능이 정상적으로 작동합니다
  • 불필요한 코드를 제거했습니다

💬 추가 설명 or 리뷰 포인트 (선택)

  • 해당 브랜치도 스쿼시 할게요

@coderabbitai
Copy link

coderabbitai bot commented Feb 15, 2026

Warning

Rate limit exceeded

@clxxrlove has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 27 minutes and 45 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

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)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch BOOK-508-feature/#272

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

DetailRecordV2ResponseDTO 따로 안없애고 그냥 두 개로 두는 걸까요?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

각자 쓰이는 곳이 있어서 이후에 리팩토링하면서 정리할게요

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

subemotion을 서버에서 내려주는 이유가 app 단에서 데이터 안갖고 받은 데이터 그대로 뿌릴려고 그랬던 거 같은데, << 이게 맞다면 SubEmotion이 어떤 String으로 구성되면 안될 것 같습니다. 정책을 명확하게 기억하는 상태가 아니라 이거 AOS쪽에 물어보고 후속 작업하시는 거 어떠실까요?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Circle 이미지를 따로 사용해야 디자인대로 나와서 이 부분은 제가 추후에 수정할게요

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

이거 일단은 재사용하긴 했는데 그냥 clip해서 해결했는데 나중에 수정하시면 그것도 지워야 할듯요

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

넵 요거는 아마 디자인 QA하면서 나올 것 같아서 그 때 제가 처리하겠습니다

Comment on lines 83 to 87
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

이거 optional 부분에 추가해둔 toPageString 으로 통일하면 더 좋을 것 같네요

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

반영완료

@doyeonk429
Copy link
Member

doyeonk429 commented Feb 15, 2026

아 토끼 리뷰 왜없나했네.. Please wait 16 minutes and 5 seconds before requesting another review. ㅋㅋ

@clxxrlove
Copy link
Member Author

@doyeonk429 아마 3개 pr 한번에 날려서 ratelimit 걸린듯요 ㅋㅋ..

@clxxrlove clxxrlove force-pushed the BOOK-508-feature/#272 branch from ad98006 to e7bd381 Compare February 15, 2026 04:37
@clxxrlove clxxrlove merged commit 80fc1c7 into develop Feb 15, 2026
2 checks passed
@clxxrlove clxxrlove deleted the BOOK-508-feature/#272 branch February 15, 2026 04:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BOOK-508/feat] 기록 수정 UI 개선

2 participants

Comments