Skip to content

fix: 장르 chip 순서 변경#864

Merged
Sadturtleman merged 1 commit into
release/1.6.3from
fix/860
Apr 11, 2026
Merged

fix: 장르 chip 순서 변경#864
Sadturtleman merged 1 commit into
release/1.6.3from
fix/860

Conversation

@Sadturtleman

@Sadturtleman Sadturtleman commented Apr 9, 2026

Copy link
Copy Markdown
Contributor

📌𝘐𝘴𝘴𝘶𝘦𝘴

📎𝘞𝘰𝘳𝘬 𝘋𝘦𝘴𝘤𝘳𝘪𝘱𝘵𝘪𝘰𝘯

📷𝘚𝘤𝘳𝘦𝘦𝘯𝘴𝘩𝘰𝘵

image

💬𝘛𝘰 𝘙𝘦𝘷𝘪𝘦𝘸𝘦𝘳𝘴

enum class의 정의 순서 변경

Summary by CodeRabbit

릴리스 노트

  • Chores
    • 내부 코드 조직을 위한 유지보수 작업 수행

참고: 이 변경사항은 사용자에게 보이는 기능이나 동작의 변화가 없습니다.

@Sadturtleman Sadturtleman self-assigned this Apr 9, 2026
@Sadturtleman Sadturtleman added 🔨 [FIX] 버그를 수정합니다. 🧙마법사 하루 labels Apr 9, 2026
@github-actions github-actions Bot requested review from devfeijoa, m6z1 and s9hn April 9, 2026 04:11
@coderabbitai

coderabbitai Bot commented Apr 9, 2026

Copy link
Copy Markdown

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: fba49f9d-3439-4eb5-8fe9-4160e5422634

📥 Commits

Reviewing files that changed from the base of the PR and between 6f59ded and 680cdcb.

📒 Files selected for processing (1)
  • feature/feed/src/main/java/com/into/websoso/feature/feed/model/NovelCategory.kt

Walkthrough

NovelCategory enum의 상수 선언 순서가 변경되었습니다. ROMANCEROMANCE_FANTASY 상수가 MODERN_FANTASY 이후로 재배치되었으며, 상수의 식별자, 파라미터 값 또는 companion object 로직에는 변경이 없습니다.

Changes

Cohort / File(s) Summary
Enum 상수 순서 변경
feature/feed/src/main/java/com/into/websoso/feature/feed/model/NovelCategory.kt
ROMANCE, ROMANCE_FANTASY enum 상수 위치를 MODERN_FANTASY 다음으로 이동. 상수의 파라미터, 로직 또는 메서드 구현에는 변경 없음.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Suggested reviewers

  • s9hn
  • m6z1
  • devfeijoa

Poem

🐰 우리의 로맨스는 순서를 바꾸며,
판타지 뒤에 자리를 옮겨 앉고,
같은 마음으로 새로운 줄에 선다.
정렬은 변했지만 본질은 그대로,
피그마와 함께 춤을 춘다! ✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed PR 제목은 장르 칩 순서 변경이라는 주요 변경사항을 명확하게 설명하고 있으며, 코드 변경 내용과 직접적으로 일치합니다.
Description check ✅ Passed PR 설명은 템플릿 구조를 따르고 있으며, 이슈 번호(#860), 스크린샷, 리뷰어를 위한 주석이 포함되어 있으나 작업 설명 항목이 비어있습니다.
Linked Issues check ✅ Passed PR은 이슈 #860의 요구사항인 Figma 디자인에 맞춘 칩 순서 변경을 구현하여 코딩 요구사항을 충족합니다.
Out of Scope Changes check ✅ Passed 모든 변경사항은 NovelCategory enum의 상수 순서 변경에만 제한되어 있으며, 이슈 #860의 범위 내에 있습니다.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/860

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.

@Sadturtleman Sadturtleman changed the base branch from develop to release/1.6.3 April 11, 2026 13:15
@Sadturtleman Sadturtleman merged commit 9eb32bd into release/1.6.3 Apr 11, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🔨 [FIX] 버그를 수정합니다. 🧙마법사 하루

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fix: 카테고리 순서 변경

2 participants