Skip to content

[hotfix] 내 방 목록 조회, 모집중인 모임방 상세조회 api response 추가 수정#158

Merged
seongjunnoh merged 2 commits intodevelopfrom
hotfix/#157/room-show-mine-and-show-recruiting-room-response-fix
Aug 6, 2025
Merged

[hotfix] 내 방 목록 조회, 모집중인 모임방 상세조회 api response 추가 수정#158
seongjunnoh merged 2 commits intodevelopfrom
hotfix/#157/room-show-mine-and-show-recruiting-room-response-fix

Conversation

@seongjunnoh
Copy link
Collaborator

@seongjunnoh seongjunnoh commented Aug 6, 2025

#️⃣ 연관된 이슈

closes #157

📝 작업 내용

157 번 이슈 참고하시면 됩니다

📸 스크린샷

image

response에 recruitingCount 추가하였습니다

image

resposne에 추천방의 roomId 값 추가하였습니다

💬 리뷰 요구사항

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요

📌 PR 진행 시 이러한 점들을 참고해 주세요

* P1 : 꼭 반영해 주세요 (Request Changes) - 이슈가 발생하거나 취약점이 발견되는 케이스 등
* P2 : 반영을 적극적으로 고려해 주시면 좋을 것 같아요 (Comment)
* P3 : 이런 방법도 있을 것 같아요~ 등의 사소한 의견입니다 (Chore)

Summary by CodeRabbit

  • 신규 기능
    • 추천 방 정보에 방 식별자(roomId)가 추가되어 더 많은 정보를 확인할 수 있습니다.
    • 내 방 목록에서 최대 모집 인원(recruitCount)과 현재 멤버 수(memberCount)가 표시됩니다.

@coderabbitai
Copy link

coderabbitai bot commented Aug 6, 2025

Walkthrough

Room 관련 API 응답 객체에 필드가 추가되었습니다. 내 모임방 목록 조회 응답에 모집 인원 및 현재 인원 필드가 추가되고, 모집중인 방 상세보기 응답의 추천 방 객체에 roomId가 포함되었습니다. 또한, 해당 roomId가 쿼리 결과에서 올바르게 할당되도록 쿼리 리포지토리 구현이 수정되었습니다.

Changes

Cohort / File(s) Change Summary
RoomRecruitingDetailViewResponse 응답 필드 추가
src/main/java/konkuk/thip/room/adapter/in/web/response/RoomRecruitingDetailViewResponse.java
RecommendRoom record에 Long roomId 필드 추가
RoomShowMineResponse 응답 필드 추가 및 Swagger 주석
src/main/java/konkuk/thip/room/adapter/in/web/response/RoomShowMineResponse.java
MyRoom record에 recruitCount, memberCount 필드 추가 및 Swagger @Schema 설명 추가
RoomQueryRepositoryImpl 쿼리 결과 매핑 수정
src/main/java/konkuk/thip/room/adapter/out/persistence/repository/RoomQueryRepositoryImpl.java
추천 방 객체에 쿼리 결과에서 roomId 값을 할당하도록 수정

Sequence Diagram(s)

sequenceDiagram
    participant Client
    participant Controller
    participant Repository

    Client->>Controller: 내 모임방 목록 조회 API 요청
    Controller->>Repository: 방 목록 조회 쿼리
    Repository-->>Controller: recruitCount, memberCount 포함된 방 목록 반환
    Controller-->>Client: recruitCount, memberCount 포함 응답 반환

    Client->>Controller: 모집중인 방 상세보기 API 요청
    Controller->>Repository: 추천 방 목록 쿼리
    Repository-->>Controller: roomId 포함된 추천 방 목록 반환
    Controller-->>Client: roomId 포함된 추천 방 응답 반환
Loading

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~7 minutes

Assessment against linked issues

Objective Addressed Explanation
내 모임방 목록 조회 api : recruitingCount 값 추가 (#157)
모집중인 방 상세보기 api : 추천 방들의 roomId 값 추가 (#157)

Suggested labels

⚡️ fix, 🧸 현준

Suggested reviewers

  • buzz0331

Poem

🐇
새로운 필드가 쏙쏙 들어왔네,
모임방 정보가 더 풍성해졌지!
roomId와 인원수, 이제 빠짐없으니
토끼도 API 응답에 미소 짓는다네.
고마워 개발자, 오늘도 한 걸음 전진!
🥕✨

Note

⚡️ Unit Test Generation is now available in beta!

Learn more here, or try it out under "Finishing Touches" below.


📜 Recent review details

Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between e47b860 and 4b4c3e8.

📒 Files selected for processing (3)
  • src/main/java/konkuk/thip/room/adapter/in/web/response/RoomRecruitingDetailViewResponse.java (1 hunks)
  • src/main/java/konkuk/thip/room/adapter/in/web/response/RoomShowMineResponse.java (1 hunks)
  • src/main/java/konkuk/thip/room/adapter/out/persistence/repository/RoomQueryRepositoryImpl.java (1 hunks)
🧰 Additional context used
🧠 Learnings (1)
📓 Common learnings
Learnt from: seongjunnoh
PR: THIP-TextHip/THIP-Server#113
File: src/main/java/konkuk/thip/recentSearch/adapter/out/persistence/RecentSearchCommandPersistenceAdapter.java:38-44
Timestamp: 2025-07-30T14:05:04.945Z
Learning: seongjunnoh는 코드 최적화 제안에 대해 구체적인 기술적 근거와 효율성 차이를 이해하고 싶어하며, 성능 개선 방식에 대한 상세한 설명을 선호한다.
Learnt from: seongjunnoh
PR: THIP-TextHip/THIP-Server#93
File: src/main/java/konkuk/thip/room/adapter/out/persistence/RoomQueryPersistenceAdapter.java:49-114
Timestamp: 2025-07-28T16:44:31.224Z
Learning: seongjunnoh는 코드 중복 문제에 대한 리팩토링 제안을 적극적으로 수용하고 함수형 인터페이스를 활용한 해결책을 선호한다.
Learnt from: seongjunnoh
PR: THIP-TextHip/THIP-Server#112
File: src/main/java/konkuk/thip/feed/adapter/out/persistence/repository/FeedQueryRepositoryImpl.java:272-272
Timestamp: 2025-07-30T10:44:34.115Z
Learning: seongjunnoh는 피드 커서 페이지네이션에서 LocalDateTime 단일 커서 방식을 선호하며, 복합 키 기반 커서보다 구현 단순성과 성능을 우선시한다.
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
  • GitHub Check: build
🔇 Additional comments (3)
src/main/java/konkuk/thip/room/adapter/out/persistence/repository/RoomQueryRepositoryImpl.java (1)

172-172: LGTM! roomId 매핑 추가가 적절합니다.

RecommendRoom 응답 객체에 새로 추가된 roomId 필드에 대한 매핑이 올바르게 구현되었습니다. 이미 쿼리의 select 절(Line 157)에서 room.roomId를 조회하고 있어 일관성 있는 변경입니다.

src/main/java/konkuk/thip/room/adapter/in/web/response/RoomRecruitingDetailViewResponse.java (1)

31-31: LGTM! RecommendRoom에 roomId 필드 추가가 적절합니다.

추천 방 객체에 식별자를 포함하여 클라이언트에서 해당 방으로 이동할 수 있도록 하는 변경사항이 PR 목표와 일치하고 올바르게 구현되었습니다.

src/main/java/konkuk/thip/room/adapter/in/web/response/RoomShowMineResponse.java (1)

18-22: 필드명 일관성 확인 완료 — recruitCount 사용이 전체 코드베이스에 걸쳐 일관됩니다.

RoomShowMineResponse에 추가된 recruitCountmemberCount 필드는 도메인(Room.java), DTO(RoomQueryDto 등), Mapper, JPA 엔티티, API 요청·응답, 테스트 전반에서 모두 recruitCount로 통일되어 있어 의도와 일치합니다. 추가 검토나 수정 없이 이대로 승인합니다.

✨ Finishing Touches
  • 📝 Generate Docstrings
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch hotfix/#157/room-show-mine-and-show-recruiting-room-response-fix

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
🪧 Tips

Chat

There are 3 ways to chat with CodeRabbit:

  • Review comments: Directly reply to a review comment made by CodeRabbit. Example:
    • I pushed a fix in commit <commit_id>, please review it.
    • Explain this complex logic.
    • Open a follow-up GitHub issue for this discussion.
  • Files and specific lines of code (under the "Files changed" tab): Tag @coderabbitai in a new review comment at the desired location with your query. Examples:
    • @coderabbitai explain this code block.
  • PR comments: Tag @coderabbitai in a new PR comment to ask questions about the PR branch. For the best results, please provide a very specific query, as very limited context is provided in this mode. Examples:
    • @coderabbitai gather interesting stats about this repository and render them as a table. Additionally, render a pie chart showing the language distribution in the codebase.
    • @coderabbitai read src/utils.ts and explain its main purpose.
    • @coderabbitai read the files in the src/scheduler package and generate a class diagram using mermaid and a README in the markdown format.

Support

Need help? Create a ticket on our support page for assistance with any issues or questions.

CodeRabbit Commands (Invoked using PR comments)

  • @coderabbitai pause to pause the reviews on a PR.
  • @coderabbitai resume to resume the paused reviews.
  • @coderabbitai review to trigger an incremental review. This is useful when automatic reviews are disabled for the repository.
  • @coderabbitai full review to do a full review from scratch and review all the files again.
  • @coderabbitai summary to regenerate the summary of the PR.
  • @coderabbitai generate docstrings to generate docstrings for this PR.
  • @coderabbitai generate sequence diagram to generate a sequence diagram of the changes in this PR.
  • @coderabbitai generate unit tests to generate unit tests for this PR.
  • @coderabbitai resolve resolve all the CodeRabbit review comments.
  • @coderabbitai configuration to show the current CodeRabbit configuration for the repository.
  • @coderabbitai help to get help.

Other keywords and placeholders

  • Add @coderabbitai ignore anywhere in the PR description to prevent this PR from being reviewed.
  • Add @coderabbitai summary to generate the high-level summary at a specific location in the PR description.
  • Add @coderabbitai anywhere in the PR title to generate the title automatically.

CodeRabbit Configuration File (.coderabbit.yaml)

  • You can programmatically configure CodeRabbit by adding a .coderabbit.yaml file to the root of your repository.
  • Please see the configuration documentation for more information.
  • If your editor has YAML language server enabled, you can add the path at the top of this file to enable auto-completion and validation: # yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json

Documentation and Community

  • Visit our Documentation for detailed information on how to use CodeRabbit.
  • Join our Discord Community to get help, request features, and share feedback.
  • Follow us on X/Twitter for updates and announcements.

@seongjunnoh seongjunnoh merged commit 6070cfd into develop Aug 6, 2025
2 checks passed
@seongjunnoh seongjunnoh deleted the hotfix/#157/room-show-mine-and-show-recruiting-room-response-fix branch August 6, 2025 09:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[THIP2025-216] [hotfix] 내 모임방 목록 조회, 모집중인 방 상세보기 api response 추가 수정

2 participants