feat: 모임 참여자 정보 조회 기능을 구현 #192
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
choidongkuen
added a commit
that referenced
this pull request
Feb 11, 2024
* feat: 리뷰 등록시 회원 매너 온도 반영 기능 추가 및 리뷰 등록 리팩토링 (#158) * refactor: 회원 리뷰 등록 API 리팩토링 (#157) * feat: 회원 리뷰 등록시, 온도 반영 및 리뷰 피드백 반영 (#157) * feat: User 엔티티 메소드 추가 (#157) * feat: Review enum 필드 추가 (#157) * test: 리뷰 등록시 온도 업데이트에 대한 단위 테스트 (#157) * test: 리뷰 등록 기능 통합 테스트 (#157) * refactor: 회원 리뷰 등록 메소드 수정 (#157) * test: 불필요한 테스트 제거 및 CI 오류 수정 (#157) * test: MeetingRepository 테스트에서 시간과 id비교 비활서화 * refactor: 회원 탈퇴 URI 변경 (#162) * refactor: 회원 탈퇴 URI 수정 (#161) * test: 회원 탈퇴 URI 수정에 대한 테스트 수정 (#161) * fix: CI 에러 수정 (#161) * fix: 필드 값 비교를 위해 deprecated 된 메소드을 대체 (#161) * fix: 필드 값 비교를 위해 deprecated 된 메소드을 대체 (#161) * feat: fcm token 업데이트 api 추가 (#166) * fix: user_alert ddl의 pk에 auto_increment를 추가한다 * feat: token 업데이트 api를 추가한다 * fix: 모임 수정 시 이미지가 추가되는 버그 수정 (#169) * perf: upgrade GPT-3.5 to GPT-4 (#174) * refactor: 위치 기반 데이터 값 유효기간 변경 * refactor: 위치 기반 데이터 값 유효기간 변경 * refactor: 위치 기반 데이터 값 유효기간 변경 * feat: 모임 종료, 사용자 추천 알림을 추가 (#182) * feat: Alert Entity를 추가하고, 알림 전송 기능이 Entity를 파라미터로 받아 동작하게 수정한다 * refactor: MeetingAlerted를 BeforeMeetingAlerted로 직관적으로 변경한다 * feat: 모임 종료 알림을 추가한다 * feat: 유저가 추천받았을때 알림 전송 기능을 추가한다 * feat: 유저가 한달 동안 받은 알람을 조회하는 기능 개발 (#185) * fix: flyway schema에 _이 하나밖에 없는 버그를 수정한다 * feat: 유저가 한달동안 받은 알람을 조회하는 기능을 개발한다 * refactor: AlertRepository의 finalAll 메소드를 findAllByCreatedAt으로 변경한다 * feat: 북마크 추가, 취소 기능 구현 (#184) * chore: 북마크 스키마 추가 * feat: 북마크 추가, 삭제 기능 * test: 북마크 추가, 삭제 테스트 작성 - 북마크 추가 케이스 - 북마크 취소 케이스 - 모임 단일 응답에 북마크 여부 추 * feat: bookmarked -> isBookmarked, V1__ 언더바 누락 수정 * fix: flyway_schema alert userId를 user_id로 변경한다 * fix: flyway 13 의 alert table문을 변경한다 * refactor: 알람 조회에 읽은알람인지, 언제 발행되었는지를 추가 (#189) * feat: 회원탈퇴, 로그아웃 시, 회원 위치 데이터 삭제 로직 추가 및 위치 데이터 유효기간 변경 (#179) * refactor: 주변 위치 데이터 응답 데이터 구조 변경 및 유효기간 30 초 설정 (#178) * feat: 회원탈퇴, 로그아웃 시 저장되어 있는 위치 정보 Redis 에서 삭제 (#178) * feat: 회원탈퇴, 로그아웃 시 저장되어 있는 위치 정보 Redis 삭제 메소드 추가 (#178) * test: 로직 수정에 따른 단위 테스트 수정 (#178) * refactor: UserService 로그아웃, 회원 탈퇴 시 위치 데이터 삭제 로직 제거 (#178) * refactor: logout @transactional 제거 (#178) * refactor: TeumTeumService 리팩토링 (#178) * test: 회원탈퇴, 로그아웃 유닛 테스트 수정 (#178) * feat: 모임 참여자 정보 조회 기능을 구현 (#192) * feat: 모임 참여자 정보 조회 API 기능 구현 (#178) * test: 테스트 코드 구현 및 테스트 (#178) --------- Co-authored-by: devxb <[email protected]> Co-authored-by: xb205 <[email protected]> Co-authored-by: ddingmin <[email protected]>
ddingmin
reviewed
Feb 11, 2024
| @@ -64,7 +64,7 @@ private String toCommaString(List<Long> ids) { | |||
| for (int i = 0; i < ids.size() - 1; i++) { | |||
| stringBuilder.append(ids.get(i)).append(","); | |||
| } | |||
| stringBuilder.append(ids.getLast()); | |||
| stringBuilder.append(ids.get(ids.size() - 1)); | |||
There was a problem hiding this comment.
범용적으로 쓸 걸 생각해서 일부만 내려주는것 보다 모두 내려주는게 좋을것 같아요 리뷰에만 한정된게 아니니까요!
There was a problem hiding this comment.
해당 응답 객체는 리뷰 생성시에만 한정해서 사용하려고 구현한 응답 클래스입니다.
굳이 모든 정보가 필요할까 해서 필요한 정보만 내려주고 있습니다!
혹시 또 사용되는 부분이 있나요?!
There was a problem hiding this comment.
그러면 네이밍을 디테일하게 가야할 것 같아요. 리뷰에 한정된 응답이니까요!
There was a problem hiding this comment.
와 네이밍까지 😭섬세한 리뷰 감사합니다 🔥
Comment on lines
+160
to
+163
| .map(userConnector::findUserById) | ||
| .flatMap(Optional::stream) | ||
| .map(MeetingParticipantsResponse::of) | ||
| .toList(); |
There was a problem hiding this comment.
map은 한 번만 사용하도록 묶어주면 좋을 것 같아요~ 옵셔널 널처리도 필요해보입니당
There was a problem hiding this comment.
flatMap(Optional::stream) 을 통해 null 을 감싸고 있는 Optional 객체는 자동으로 무시되는데,
모임 참여하고 리뷰 등록 사이에 회원 탈퇴을 하는 경우를 생각해서 그냥 Optional 은 무시하도록 로직을 구현했습니다!
| @@ -97,6 +98,12 @@ public void deleteParticipant(@PathVariable("meetingId") Long meetingId) { | |||
| meetingService.cancelParticipant(meetingId, userId); | |||
| } | |||
|
|
|||
| @GetMapping("/{meetingId}/participants") | |||
| @ResponseStatus(HttpStatus.OK) | |||
| public List<MeetingParticipantsResponse> getParticipants(@PathVariable("meetingId") Long meetingId) { | |||
|
|
||
| import net.teumteum.user.domain.User; | ||
|
|
||
| public record MeetingParticipantsResponse( |
There was a problem hiding this comment.
Suggested change
| public record MeetingParticipantsResponse( | |
| public record MeetingParticipantResponse( |
단수입니당!
There was a problem hiding this comment.
모임에 참여한 회원"들"의 정보라 s 을 붙인건데 단수로 취급해야하나요~?!
There was a problem hiding this comment.
Dto에 포함된 내용들은 유저에 해당하기때문에 단수로 표현해야할 것 같습니당
There was a problem hiding this comment.
아 맞네요 그럼 DTO 로 변환해서 반환하도록 수정하겠습니다!
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
🚀 어떤 기능을 개발했나요?
모임 id 에 해당하는 모임 참여한 회원 정보 조회
🕶️ 어떻게 해결했나요?
🦀 이슈 넘버