Skip to content

refactor/#206 북마크 개수 제한 제거#207

Merged
woals2840 merged 1 commit into
developfrom
refactor/#206
Jun 9, 2026
Merged

refactor/#206 북마크 개수 제한 제거#207
woals2840 merged 1 commit into
developfrom
refactor/#206

Conversation

@woals2840

@woals2840 woals2840 commented Jun 9, 2026

Copy link
Copy Markdown
Contributor

📣 Related Issue

  • close #

📝 Summary

🙏 Question & PR point

📬 Postman

Summary by CodeRabbit

릴리스 노트

  • 버그 수정
    • 북마크된 콘텐츠 조회 로직을 개선했습니다. 이제 고정된 개수 제한 없이 전체 북마크 목록을 조회할 수 있으며, 필요에 따라 유연한 제한 조건을 적용할 수 있습니다.

@woals2840 woals2840 linked an issue Jun 9, 2026 that may be closed by this pull request
1 task
@coderabbitai

coderabbitai Bot commented Jun 9, 2026

Copy link
Copy Markdown

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: ebd09db3-deb3-486b-808d-8d625a4d10f0

📥 Commits

Reviewing files that changed from the base of the PR and between 4565fae and aa4d699.

📒 Files selected for processing (1)
  • apps/api/src/main/java/kr/flint/api/domain/content/repository/ContentQueryRepository.java

📝 Walkthrough

Walkthrough

ContentQueryRepository에서 북마크 작품 조회 로직을 변경했습니다. getBookmarkedContentRows 메서드가 limit 파라미터를 조건부로 처리하도록 수정되어, limit > 0일 때만 쿼리에 제한을 적용하고 그 이하일 때는 전체 결과를 반환합니다. getContentDetailList 메서드는 이제 getBookmarkedContentRows(userId, null, 0)을 호출하여 모든 북마크된 작품을 조회합니다(기존의 고정 10개 조회에서 변경).

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

북마크 수를 한계 없이,
limit 영 파라미터로 우아하게 👌
조건의 논리로 유연함을 얻고,
작품 목록이 한껏 펼쳐지네 📚✨

✨ 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 refactor/#206

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.

@woals2840
woals2840 merged commit f656701 into develop Jun 9, 2026
1 check was pending
@woals2840
woals2840 deleted the refactor/#206 branch June 9, 2026 14:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[REFACTOR] 북마크 개수 제한 수정

1 participant