Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[feat #122] 채팅방 목록 조회 API 필터링 추가 #123

Merged
merged 7 commits into from
Sep 30, 2024
Merged

[feat #122] 채팅방 목록 조회 API 필터링 추가 #123

merged 7 commits into from
Sep 30, 2024

Conversation

hyun2371
Copy link
Member

관련 이슈

📑 작업 상세 내용

  • 기존 채팅방 목록 조회 API에 필터링 추가
    • 채팅방 상태(요청중, 수락됨)에 따라 분류
    • request param으로 채팅방 상태 받기

💫 작업 요약

  • 필터링 추가

@hyun2371 hyun2371 added the ✨ feat 기능 추가 label Sep 28, 2024
@hyun2371 hyun2371 requested a review from dudxo September 28, 2024 07:52
@hyun2371 hyun2371 self-assigned this Sep 28, 2024
@hyun2371 hyun2371 linked an issue Sep 28, 2024 that may be closed by this pull request
✨ 채팅방 목록 조회 API 필터링 추가 #122
Closed
1 task
@github-actions GitHub Actions
Copy link

github-actions bot commented Sep 28, 2024 •
edited
Loading

Code Coverage

Overall Project 86.33% -0.12% 🍏
Files changed 88.89% 🍏

File Coverage
ChatRoomQueryRepositoryImpl.java 100% 🍏
ChatRoomController.java 100% 🍏
ChatRoomService.java 91.03% 🍏
ChatStatus.java 89.58% -10.42% 🍏

@github-actions GitHub Actions
Copy link

github-actions bot commented Sep 28, 2024 •
edited
Loading

Test Results

 25 files   25 suites   15s ⏱️
115 tests 115 ✅ 0 💤 0 ❌
116 runs  116 ✅ 0 💤 0 ❌

Results for commit fc569a7.

♻️ This comment has been updated with latest results.

dudxo
dudxo approved these changes Sep 29, 2024
View reviewed changes
Copy link
Collaborator

@dudxo dudxo left a comment

Choose a reason for hiding this comment

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

고생하셨습니다!
토~일 해커톤 행사 참여하느라 리뷰가 늦었네요,,
코드를 쭉 보니 채팅방 목록 조회 API에서 요청중 || 수락됨 || 거절됨 총 3가지 필터 값을 통해서 조회하는 로직을 추가하신 것 같습니다. 피그마와 지난 주 슬랙에 올라온 업데이트 된 UI들을 확인해봤는데 기존 필터링 기능이 었었나요?

따로 없는데 추가하신 이유가 궁금합니다!

src/main/java/com/dnd/gongmuin/chat/exception/ChatErrorCode.java Outdated
@@ -13,7 +13,9 @@ public enum ChatErrorCode implements ErrorCode {
NOT_FOUND_CHAT_ROOM("해당 아이디의 채팅방이 존재하지 않습니다.", "CH_002"),
UNAUTHORIZED_REQUEST("채팅 수락을 하거나 거절할 권한이 없습니다.", "CH_003"),
UNABLE_TO_CHANGE_CHAT_STATUS("이미 수락했거나 거절한 요청입니다.", "CH_004"),
UNAUTHORIZED_CHAT_ROOM("권한이 없는 채팅방입니다", "CH_005");
UNAUTHORIZED_CHAT_ROOM("권한이 없는 채팅방입니다.", "CH_005"),

Copy link
Collaborator

Choose a reason for hiding this comment

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

사소하지만 줄바꿈이 한 줄 되어있네요..!

@hyun2371
Copy link
Member Author

고생하셨습니다! 토~일 해커톤 행사 참여하느라 리뷰가 늦었네요,, 코드를 쭉 보니 채팅방 목록 조회 API에서 요청중 || 수락됨 || 거절됨 총 3가지 필터 값을 통해서 조회하는 로직을 추가하신 것 같습니다. 피그마와 지난 주 슬랙에 올라온 업데이트 된 UI들을 확인해봤는데 기존 필터링 기능이 었었나요?

따로 없는데 추가하신 이유가 궁금합니다!

image

요청중 파라미터를 넘길 때 요청 목록, 수락됨 파라미터를 넘길 때 채팅 목록을 반환할 수 있도록 설계했습니다!
추가적으로 거절됨이나 그 외 파라미터값을 받으면 예외를 반환하도록 하였습니다~

@dudxo
Copy link
Collaborator

dudxo commented Sep 30, 2024

image

요청중 파라미터를 넘길 때 요청 목록, 수락됨 파라미터를 넘길 때 채팅 목록을 반환할 수 있도록 설계했습니다! 추가적으로 거절됨이나 그 외 파라미터값을 받으면 예외를 반환하도록 하였습니다~

아하, 채팅 목록과 요청 목록에서는 해당 필터링들이 필요하겠네요! 설명 감사합니다 ㅎㅎ

@dudxo
Copy link
Collaborator

dudxo commented Sep 30, 2024

image

요청중 파라미터를 넘길 때 요청 목록, 수락됨 파라미터를 넘길 때 채팅 목록을 반환할 수 있도록 설계했습니다! 추가적으로 거절됨이나 그 외 파라미터값을 받으면 예외를 반환하도록 하였습니다~

아하, 채팅 목록과 요청 목록에서는 해당 필터링들이 필요하겠네요! 설명 감사합니다 ㅎㅎ

@hyun2371 hyun2371 merged commit b1d7de8 into dev Sep 30, 2024
3 checks passed
@hyun2371 hyun2371 deleted the feat/#122/chat-rooms-status-filter branch October 2, 2024 12:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dudxo dudxo dudxo approved these changes

Assignees

@hyun2371 hyun2371

Labels
✨ feat 기능 추가
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

✨ 채팅방 목록 조회 API 필터링 추가
2 participants
@hyun2371 @dudxo