feat: 리뷰 등록시 요청 DTO Validation 변경 및 로직 추가 #233
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 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 <ddingminyt@gmail.com> Co-authored-by: devxb <dlwnsdud205@naver.com> Co-authored-by: xb205 <62425964+devxb@users.noreply.github.com>
ddingmin
added a commit
that referenced
this pull request
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 <danaver12@daum.net> Co-authored-by: devxb <dlwnsdud205@naver.com> Co-authored-by: xb205 <62425964+devxb@users.noreply.github.com>
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.
🚀 어떤 기능을 개발했나요?
🕶️ 어떻게 해결했나요?
min = 1로 변경 (기획에 대한 잘못된 이해 이슈)🦀 이슈 넘버