Skip to content

fix: 서재 datastore sorttype 마이그레이션 - #927

Merged
Sadturtleman merged 1 commit into
developfrom
fix/926
Jul 24, 2026
Merged

fix: 서재 datastore sorttype 마이그레이션#927
Sadturtleman merged 1 commit into
developfrom
fix/926

Conversation

@Sadturtleman

@Sadturtleman Sadturtleman commented Jul 24, 2026

Copy link
Copy Markdown
Contributor

📌𝘐𝘴𝘴𝘶𝘦𝘴

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

📷𝘚𝘤𝘳𝘦𝘦𝘯𝘴𝘩𝘰𝘵

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

https://discord.com/channels/1185753783020568648/1530072575114346506 참고하시면 될 것 같습니다

Summary by CodeRabbit

  • 버그 수정
    • 기존에 저장된 정렬 기준을 최신 정렬 방식으로 자동 변환합니다.
    • 지원되지 않거나 잘못된 정렬 기준은 기본 정렬 기준으로 보정됩니다.

@Sadturtleman Sadturtleman added the 🔨 [FIX] 버그를 수정합니다. label Jul 24, 2026
@github-actions
github-actions Bot requested review from devfeijoa, m6z1 and s9hn July 24, 2026 06:04
@coderabbitai

coderabbitai Bot commented Jul 24, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 438e75c9-434f-4d31-99a7-a867e15a5fd0

📥 Commits

Reviewing files that changed from the base of the PR and between 2cb8235 and 16810d4.

📒 Files selected for processing (1)
  • core/datastore/src/main/java/com/into/websoso/core/datastore/datasource/library/mapper/LibraryFilterMapper.kt

Walkthrough

라이브러리 필터의 레거시 정렬 기준을 v2 키로 마이그레이션하고, 허용되지 않은 값은 기본값으로 정규화하도록 변경했습니다.

Changes

라이브러리 정렬 기준 마이그레이션

Layer / File(s) Summary
정렬 기준 변환 및 검증
core/datastore/.../LibraryFilterMapper.kt
RECENTOLD를 v2 정렬 키로 변환하고, 허용 목록에 없는 값은 DEFAULT_SORT_CRITERIA로 대체한 뒤 LibraryFilter에 전달합니다.

Estimated code review effort: 2 (Simple) | ~10 minutes

Poem

깡충 토끼가 정렬을 살펴
오래된 키는 새 길로 보내고
낯선 값은 기본값에 포근히
서재의 책들이 다시 보이네
당근처럼 깔끔한 변환! 🐇

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Description check ❓ Inconclusive 템플릿의 주요 섹션은 갖췄지만 Work Description과 Screenshot 내용이 비어 있어 설명이 부족합니다. Work Description에 변경 내용과 영향 범위를 채우고, 스크린샷이 없다면 생략 사유를 적어주세요.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed 제목이 서재 datastore sorttype 마이그레이션 수정이라는 핵심 변경을 정확히 요약합니다.
Linked Issues check ✅ Passed sortCriteria 마이그레이션과 기본값 적용이 내 서재 작품 미노출 문제(#926) 해결 목적과 맞습니다.
Out of Scope Changes check ✅ Passed 변경은 LibraryFilterMapper의 sortCriteria 마이그레이션에만 국한되어 목적 외 수정은 보이지 않습니다.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ 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/926

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.

@Sadturtleman
Sadturtleman merged commit 88a1b48 into develop Jul 24, 2026
3 checks passed
@Sadturtleman
Sadturtleman deleted the fix/926 branch July 24, 2026 09:05
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