-
Notifications
You must be signed in to change notification settings - Fork 1
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.
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
feat: 유저가 받은 리뷰 조회 로직 변경 #217
Labels
Comments
choidongkuen
added a commit
that referenced
this issue
Feb 13, 2024
choidongkuen
added a commit
that referenced
this issue
Feb 13, 2024
choidongkuen
added a commit
that referenced
this issue
Feb 13, 2024
devxb
added a commit
that referenced
this issue
Feb 15, 2024
* fix: 충돌 해결 (#210) * fix: 소셜 로그인 에러 긴급 수정 (#214) (#215) * feat: 기존 회원 리뷰 조회 API 로직 변경 (#218) * feat: userId 에 해당하는 유저 리뷰 조회 API 로직 변경 (#217) * test: 통합테스트 수정 (#217) * test: 단위테스트 수정 (#217) * docs: readme 작성 (#222) * docs: readme 작성 * docs: readme 수정 * test: QA용 EndMeetingAlerted를 발행한다 --------- Co-authored-by: ddingmin <[email protected]> Co-authored-by: ChoiDongKuen <[email protected]>
devxb
added a commit
that referenced
this issue
Feb 15, 2024
* fix: 충돌 해결 (#210) * fix: 소셜 로그인 에러 긴급 수정 (#214) (#215) * feat: 기존 회원 리뷰 조회 API 로직 변경 (#218) * feat: userId 에 해당하는 유저 리뷰 조회 API 로직 변경 (#217) * test: 통합테스트 수정 (#217) * test: 단위테스트 수정 (#217) * docs: readme 작성 (#222) * docs: readme 작성 * docs: readme 수정 * test: QA용 EndMeetingAlerted를 발행한다 * feat: 유저가 삭제되었을때, 알람토큰도 삭제하도록 수정 (#226) * feat: 유저가 삭제되었을때, 알람토큰도 삭제하도록 수정한다 * test: ApplicationEventPublisher 바인딩 안되는 테스트 삭제 * refactor: UserDeletedEvent 핸들러 이름 변경 * test: 친구조회 테스트 수정 * refactor: code smell을 제거한다 --------- Co-authored-by: ddingmin <[email protected]> Co-authored-by: ChoiDongKuen <[email protected]>
devxb
added a commit
that referenced
this issue
Feb 16, 2024
* fix: 충돌 해결 (#210) * fix: 소셜 로그인 에러 긴급 수정 (#214) (#215) * feat: 기존 회원 리뷰 조회 API 로직 변경 (#218) * feat: userId 에 해당하는 유저 리뷰 조회 API 로직 변경 (#217) * test: 통합테스트 수정 (#217) * test: 단위테스트 수정 (#217) * docs: readme 작성 (#222) * docs: readme 작성 * docs: readme 수정 * test: QA용 EndMeetingAlerted를 발행한다 * feat: 유저가 삭제되었을때, 알람토큰도 삭제하도록 수정 (#226) * feat: 유저가 삭제되었을때, 알람토큰도 삭제하도록 수정한다 * test: ApplicationEventPublisher 바인딩 안되는 테스트 삭제 * refactor: UserDeletedEvent 핸들러 이름 변경 * test: 친구조회 테스트 수정 * refactor: code smell을 제거한다 * fix: 모임 시작, 종료 알림이 모임 참여자가 3명 이상이 아니라면, 발행되지않도록 수정 (#231) * feat: 리뷰 등록시 모임 참여 유저 조회 API 수정 (#229) * refactor: MeetingController getCurrentUserId() 메소드로 통일 (#228) * feat: 리뷰에 자신 포함 확인 로직 추가 (#195) * feat: 모임 참여자 자신 포함 여부 확인 로직 추가(#228) * test: 단위 테스트 추가 (#228) * test: 통합 테스트 수정 및 실패 케이스 추가 (#228) * refactor: title, body alert data에 추가 --------- Co-authored-by: ddingmin <[email protected]> Co-authored-by: ChoiDongKuen <[email protected]>
choidongkuen
added a commit
that referenced
this issue
Feb 16, 2024
* fix: 충돌 해결 (#210) * fix: 소셜 로그인 에러 긴급 수정 (#214) (#215) * feat: 기존 회원 리뷰 조회 API 로직 변경 (#218) * feat: userId 에 해당하는 유저 리뷰 조회 API 로직 변경 (#217) * test: 통합테스트 수정 (#217) * test: 단위테스트 수정 (#217) * docs: readme 작성 (#222) * docs: readme 작성 * docs: readme 수정 * test: QA용 EndMeetingAlerted를 발행한다 * feat: 유저가 삭제되었을때, 알람토큰도 삭제하도록 수정 (#226) * feat: 유저가 삭제되었을때, 알람토큰도 삭제하도록 수정한다 * test: ApplicationEventPublisher 바인딩 안되는 테스트 삭제 * refactor: UserDeletedEvent 핸들러 이름 변경 * test: 친구조회 테스트 수정 * refactor: code smell을 제거한다 * fix: 모임 시작, 종료 알림이 모임 참여자가 3명 이상이 아니라면, 발행되지않도록 수정 (#231) * feat: 리뷰 등록시 모임 참여 유저 조회 API 수정 (#229) * refactor: MeetingController getCurrentUserId() 메소드로 통일 (#228) * feat: 리뷰에 자신 포함 확인 로직 추가 (#195) * feat: 모임 참여자 자신 포함 여부 확인 로직 추가(#228) * test: 단위 테스트 추가 (#228) * test: 통합 테스트 수정 및 실패 케이스 추가 (#228) * refactor: title, body alert data에 추가 * feat: 리뷰 등록시 요청 DTO Validation 변경 및 로직 추가 (#233) * feat: 요청 validation 변경 (#228) * test: UserControllerTest 리팩토링 (#224) * feat: 리뷰 등록시 userService 로직 추가 (#224) * test: 테스트 데이터 관련 메소드 추가 (#224) * test: 리뷰 등록시 로직 추가에 따른 통합 테스트 추가 (#224) * test: 리뷰 등록시 로직 추가에 따른 Service 단위 테스트 추가 (#224) * feat: Cors 허용 메소드 추가 (#224) --------- Co-authored-by: ddingmin <[email protected]> Co-authored-by: devxb <[email protected]> Co-authored-by: xb205 <[email protected]>
ddingmin
added a commit
that referenced
this issue
Feb 16, 2024
* fix: 충돌 해결 (#210) * fix: 소셜 로그인 에러 긴급 수정 (#214) (#215) * feat: 기존 회원 리뷰 조회 API 로직 변경 (#218) * feat: userId 에 해당하는 유저 리뷰 조회 API 로직 변경 (#217) * test: 통합테스트 수정 (#217) * test: 단위테스트 수정 (#217) * docs: readme 작성 (#222) * docs: readme 작성 * docs: readme 수정 * test: QA용 EndMeetingAlerted를 발행한다 * feat: 유저가 삭제되었을때, 알람토큰도 삭제하도록 수정 (#226) * feat: 유저가 삭제되었을때, 알람토큰도 삭제하도록 수정한다 * test: ApplicationEventPublisher 바인딩 안되는 테스트 삭제 * refactor: UserDeletedEvent 핸들러 이름 변경 * test: 친구조회 테스트 수정 * refactor: code smell을 제거한다 * fix: 모임 시작, 종료 알림이 모임 참여자가 3명 이상이 아니라면, 발행되지않도록 수정 (#231) * feat: 리뷰 등록시 모임 참여 유저 조회 API 수정 (#229) * refactor: MeetingController getCurrentUserId() 메소드로 통일 (#228) * feat: 리뷰에 자신 포함 확인 로직 추가 (#195) * feat: 모임 참여자 자신 포함 여부 확인 로직 추가(#228) * test: 단위 테스트 추가 (#228) * test: 통합 테스트 수정 및 실패 케이스 추가 (#228) * refactor: title, body alert data에 추가 * feat: 리뷰 등록시 요청 DTO Validation 변경 및 로직 추가 (#233) * feat: 요청 validation 변경 (#228) * test: UserControllerTest 리팩토링 (#224) * feat: 리뷰 등록시 userService 로직 추가 (#224) * test: 테스트 데이터 관련 메소드 추가 (#224) * test: 리뷰 등록시 로직 추가에 따른 통합 테스트 추가 (#224) * test: 리뷰 등록시 로직 추가에 따른 Service 단위 테스트 추가 (#224) * feat: Cors 허용 메소드 추가 (#224) * feat: 위치 기반 API Gatling 부하 테스트 (#220) * chore: docker-compose.yml 작성 (#195) * test: Gatling 부하 테스트 구현 (#195) * Update .env * Delete .env --------- Co-authored-by: ChoiDongKuen <[email protected]> Co-authored-by: devxb <[email protected]> Co-authored-by: xb205 <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
기능
Todo
The text was updated successfully, but these errors were encountered: