develop 브랜치에 develop-be 브랜치 최신화 #570
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
* !hotfix: 버그 리포팅을 위한 이슈 템플릿 추가 * !hotfix: 버그 리포팅용 이슈 템플릿의 설명을 조금 더 명확하게 변경 * !hotfix: 개발 도중 버그가 발생했을 때의 이슈 템플릿의 설명을 명확하게 변경 * refactor: 메시지 조회 시 연관 엔티티 fetch join하도록 변경 * style: 120자 넘어가는 줄 개행 * refactor: 메시지 조회 dto 필드 변경 * refactor: 불필요한 fetch join 제거 * refactor: 불필요한 user 조회 로직 제거 * refactor: 불필요한 fetch join 제거 및 조회한 chatRoom DTO로 전달 * refactor: 조회한 채팅방 엔티티를 DTO변환에 사용하도록 변경 --------- Co-authored-by: 지토 <[email protected]>
* feat: api에서 전달된 정렬 조건을 필드 명으로 변경하는 기능 추가 * feat: Controller에서 기본 정렬 방향이 DESC인 Pageable을 설정하는 ArgumentResolver 추가 * refactor: 경매 목록 조회 시 int가 아닌 Pageable을 전달받도록 변경 * rename: api에서 전달된 정렬 조건을 필드 명으로 변경하는 기능 네이밍 변경 * rename: api에서 전달된 정렬 조건을 필드 명으로 변경하는 기능 네이밍 변경 * feature: 정렬 조건을 Querydsl 표현식으로 변환하는 기능 추가 * feat: 경매 목록 조회 시 정렬 관련 쿼리가 실행되는 기능 추가 * test: 누락된 예외 테스트 케이스 추가 * fix: 누락된 정렬 조건 추가 * test: 메서드 변경으로 인해 실패하는 테스트 케이스 수정 * refactor: Converter 과정 중 일부 로직 메서드 분리 * rename: 기본 정렬 조건을 내림차순으로 설정하는 ArgumentResolver 네이밍 변경 * style: return 개행 추가 * style: final 키워드 추가 * style: 불필요한 import 코드 제거 * fix: ArgumentResolver에서 관리하던 기본 값을 enum으로 분리 * refactor: QuerydslSliceHelper 메서드 네이밍 변경 * refactor: enum Converter 네이밍 수정 * refactor: enum Converter 공통되는 문자열 상수 분리 * refactor: 정렬 시 항상 auction의 id를 내림차순으로 정렬하도록 조건 추가 * test: 누락된 테스트 케이스 추가
* feat: 경매 목록 검색 DTO 추가 * feat: 경매 목록 조회 시 제목 검색 기능 추가 * rename: 경매 목록 조회 시 검색 DTO SearchCondition 네이밍 변경 * feat: ReadAuctionSearchCondition ArgumentResolver 관련 기능 추가 * test: 경매 목록 조회 시 제목 검색에 대한 더미 데이터와 테스트 케이스 추가 * test: 경매 목록 조회 시 제목 검색에 대한 더미 데이터와 테스트 케이스 추가
* feat: ArgumentResolver Pageable 관련 설정 추가 * fix: no-offset 기반에서 offset 기반의 쿼리로 변경 * test: 실패하는 테스트 케이스 수정 * fix: 메서드 파라미터에서 lastAuctionId 제거 * refactor: 경매 목록 조회 메서드 네이밍 변경 * fix: 마감 임박순 정렬 내림차순에서 오름차순으로 변경 * test: 경매 목록 조회 테스트 시 더미 데이터 및 테스트 케이스 추가 * test: 잘못된 테스트 케이스 수정 * refactor: 경매 목록 요청 시 정렬 쿼리 파라미터 이름 변경
* refactor: 코드 개행, 네이밍 수정 및 로직 개선 * refactor: AuctionService 내의 ChatRoomService 의존성 분리 * perf: 레포지토리에서 단일 채팅방 조회 시의 쿼리 최적화 * refactor: 채팅방을 마지막 메시지가 최근인 순으로 정렬하는 기능에 대해 서비스에서 정렬하지 않고 레포지토리에서 정렬해서 조회하도록 변경 * perf: 레포지토리에서 마지막 메시지가 최근인 순으로 정렬하여 조회할 때 채팅방과 메시지를 함께 하나의 dto로 받아오도록 성능 개선 * test: 채팅방 목록 조회 테스트 데이터 추가 * refactor: 채팅방 목록 조회 시 반환하는 dto에 `@QueryProjection` 사용하도록 변경 * rename: 레포지토리에서 채팅방 목록 조회시 반환하는 dto 이름 변경 * perf: db에서 정렬하지 않고 레포지토리에서 정렬하도록 변경 * refactor: dto를 반환하는 레포지토리 엔티티를 반환하는 레포지토리 분리 * refactor: 불필요한 orderBy 제거 및 사용하지 않는 메서드 제거 * style: 개행 수정 * style: 개행 수정 및 불필요한 import 제거 * refactor: 리뷰 피드백 적용 - 예외 네이밍 변경 - 개행 수정 - 쿼리 간소화 * refactor: 통일성을 위해 Querydsl 키워드 명시하도록 변경
* feat: 회원이 등록한 경매 목록을 조회하는 쿼리 추가 * feat: 회원이 등록한 경매 목록을 조회하는 기능 추가 * feat: 회원이 등록한 경매 목록을 조회하는 api 추가 * feat: 회원이 참여한 경매 목록을 조회하는 쿼리 추가 * test: 테스트 클래스 네이밍 변경 * feat: 회원이 참여한 경매 목록을 조회하는 기능 추가 * feat: 회원이 참여한 경매 목록을 조회하는 api 추가 * docs: 문서 최신화 * style: 불필요한 개행 제거 * style: 메서드 순서 변경 * refactor: 사용자가 호출하는 경매 목록 조회 Controller 분리 * docs: 문서 최신화
* chore: FCM 사용을 위한 설정 추가 Co-authored-by: swonny <[email protected]> * feat: 사용자 디바이스 토큰 엔티티 및 레포지토리 추가 Co-authored-by: swonny <[email protected]> * feat: 알림 전송 서비스 구현 Co-authored-by: swonny <[email protected]> * feat: 로그인 시 알림을 위한 디바이스 토큰 저장 기능 추가 Co-authored-by: swonny <[email protected]> * feat: 알림 데이터에 알림 타입 추가 Co-authored-by: swonny <[email protected]> * feat: 메시지 전송 시 알림 전송 기능 추가 Co-authored-by: swonny <[email protected]> * refactor: 알림 속성 패키지 이동 Co-authored-by: swonny <[email protected]> * refactor: 디바이스 토큰 패키지 분리 Co-authored-by: swonny <[email protected]> * feat: 디바이스 토큰 갱신을 위한 서비스 및 api 추가 Co-authored-by: swonny <[email protected]> * refactor: 로그인 시 디바이스 토큰 저장할 때 서비스를 호출하도록 변경 Co-authored-by: swonny <[email protected]> * docs: 디바이스 토큰 관련 api 문서화 Co-authored-by: swonny <[email protected]> * feat: 알림 전송 실패 예외 핸들링 추가 Co-authored-by: swonny <[email protected]> * style: 개행 수정 및 불필요한 주석 제거 Co-authored-by: swonny <[email protected]> * fix: 사용자 기기 토큰을 토큰 아이디로 찾아서 생기는 버그 수정 Co-authored-by: swonny <[email protected]> * feat: FCM 빈 등록을 설정 파일에서 제어할 수 있도록 기능 추가 Co-authored-by: swonny <wonny [email protected]> * refactor: mock 객체의 equals() and hashCode() 재정의를 롬복 사용하도록 수정 * refactor: 토큰 갱신 api의 request method를 put -> patch로 변경 * feat: 엔티티의 기기 토큰 필드가 null이 될 수 없도록 변경 및 기기 토큰이 비어있을 경우 저장하지 않는 기능 추가 * refactor: 토큰 갱신 서비스 메서드 이름 수정 및 세부 로직 수정 * rename: 디바이스 토큰 갱신 시 application 레이어에서 사용하는 dto 이름 변경 * test: 필요없는 테스트 삭제 및 실패하는 테스트 수정 * docs: 문서 최신화 * ci: 충돌 해결 * style: 개행 수정 및 체이닝 정렬 * fix: 기기 토큰 엔티티의 toString 대상에서 lazy 로딩인 엔티티 제외 * style: enum 나열 방식 수정 * feat: 기기 토큰 엔티티에 외래키 이름 설정 추가 * feat: 기기 토큰 테이블 관련 flyway 스크립트 추가 * refactor: 알림을 위한 빈 등록 실패 시 예외를 발생시키도록 변경 * refactor: 알림 전송 실패 시 예외를 던지지 않고 실패 메시지를 반환하도록 변경 * refactor: 기기 토큰 관련 api url 변경 Co-authored-by: swonny <[email protected]> --------- Co-authored-by: kwonyj1022 <[email protected]> Co-authored-by: swonny <wonny [email protected]>
Co-authored-by: swonny <[email protected]>
* feat: 이미지 url을 만드는 util 클래스 추가 * feat: User의 이미지를 엔티티 객체로 수정 * feat: 사용자 정보 수정 기능 추가 * feat: 이미지 조회 서비스 추가 * refactor: 메시지의 이미지 url과 관련해 util을 사용하도록 수정 * refactor: 이미지를 반환하도록 수정 * feat: 사용자 프로필 이미지 조회 컨트롤러 추가 * refactor: 업데이트 메서드에 Transactional 설정 * test: userService 예외 테스트 추가 * feat: 누락된 dto 파일 추가 * refactor: 메서드명이 명확하도록 수정 * refactor: Embedded를 통한 중복 필드 제거 * rename: 이미지 url 생성 util 클래스 패키지 위치 및 네이밍 변경 * refactor: 메서드명을 명확하도록 수정 * refactor: 개행 수정 * refactor: 변수명을 컨벤션에 맞도록 수정 * refactor: 변수명 수정 및 중복 제거 * feat: 사용자 수정 컨트롤러 기능 추가 * refactor: 메서드 로직을 명확하게 분리 * ci: 충돌 문제 해결 * ci: flyway profileImage 연관관계 수정 * docs: 문서 최신화 * style: 해결된 todo 제거 * ci: flyway 문법 수정 * refactor: User profileImage의 fk 설정 --------- Co-authored-by: apptie <[email protected]>
* feat: 카카오 연결 끊기 기능 추가 * feat: 탈퇴 기능 서비스 추가 * feat: oauth id에 대한 unique 속성 제거 * refactor: 회원과 사용자라는 용어 통일 * feat: 이름이 이미 존재하는지에 대한 확인 쿼리 메서드 추가 * feat: 랜덤 이름을 생성하는 util 클래스 추가 * feat: 재가입 시 이름에 대한 중복 문제 해결을 위한 로직 추가 * feat: 탈퇴한 회원의 이름을 가져오는 경우에 대한 로직 추가 * feat: 탈퇴 컨트롤러 기능 추가 * feat: 예외처리 추가 * test: 테스트 실패 문제 해결 * refactor: flyway 버전 수정 * refactor: 탈퇴한 사용자 이름 변경 로직 위치 수정 * docs: 문서 최신화 * refactor: 예외 메시지 클래스명 수정 * refactor: do-while문을 while문으로 수정 * refactor: 탈퇴 시 로직 순서 변경 * style: 해결된 todo 제거 * ci: flyway 버전 수정 * ci: 충돌 문제 해결 * refactor: 이미지가 null인 경우에 대한 예외처리 추가 * refactor: util 클래스에 final 추가 * fix: 경매 이미지 url 경로 누락 문제 해결
* feat: 알림 전송 실패 시 로그 추가 * fix: 이미지 null 처리 문제 해결
* fix: 경매 이미지 url 처리에 방법 변경 * test: 테스트 실패 문제 해결
* refactor: 요청 메서드 타입 변경 * test: 테스트 실패 해결
* fix: 이미지 개수만큼 채팅방이 중복 조회 되는 문제 해결 * rename: dto 이름 변경 * refactor: 채팅방 아이디로 채팅방 조회할 때 쿼리 개선 * style: 나중에 하기로 한 것 todo 추가
* feat: 로그인 시 프로필 이미지를 null이 아닌 기본이미지로 세팅하는 기능 추가 * feat: 사용자 엔티티의 프로필 이미지에 널 불가 제약조건 추가 * feat: 알림 dto에 `@NonNull` 제약 조건 추가 * refactor: 알림 전송 성공 및 예외 발생 시 로그 추가 * refactor: 기본 프로필 이미지 조회 시 이름으로 조회하도록 변경 * feat: 로그인 시 기본이미지를 찾을 수 없으면 예외가 발생하는 기능 추가 * refactor: 프로필 이미지 경로 생성을 유틸 클래스에서 하도록 변경 * refactor: 기본 프로필 이미지 이름을 엔티티에서 관리하도록 변경 * fix: flyway 스크립트 수정 * refactor: 알림 dto에서 사용하는 `@NonNull`을 롬복의 어노테이션을 사용하도록 변경
* test: AuctionTest Fixture 생성 및 테스트 케이스 리펙토링 * test: BidUnitTest Fixture 생성 및 테스트 케이스 리펙토링 * test: PriceTest Fixture 생성 및 테스트 케이스 리펙토링 * test: SortParameterTest Fixture 생성 및 테스트 케이스 리펙토링 * test: AuctionServiceTest Fixture 생성 및 테스트 케이스 리펙토링 * test: AuctionControllerTest Fixture 생성 및 테스트 케이스 리펙토링 * test: QuerydslAuctionAndImageRepositoryImplTest Fixture 생성 및 테스트 케이스 리펙토링 * test: QuerydslAuctionRepository Fixture 생성 및 테스트 케이스 리펙토링 * test: JpaAuctionRepositoryTest Fixture 테스트 케이스 리펙토링 * test: JpaAuctionRepositoryTest Fixture 추가 및 테스트 케이스 리펙토링 * test: 사용하지 않는 Fixture 삭제 * test: Fixture 및 테스트 케이스 리펙토링 * test: 의미 없는 mocking 삭제 * test: 클래스 확장 시 개행 제거 * test: Fixture를 사용하지 않은 부분을 사용하도록 수정 * test: 도메인 엔티티 테스트 환경에서 JPA 제거 * test: AuctionServiceFixture 변경 및 AuctionServiceTest 테스트 케이스 수정 * test: AuctionControllerFixture 변경 및 AuctionControllerTest 테스트 케이스 수정 * refactor: TODO 삭제 * test: AuctionControllerTest Fixture 추가 및 검증 케이스 추가 * test: auction persistence 영역 테스트 어노테이션 변경 및 메서드, Fixture 네이밍을 명확하게 변경
* feat: 알림 전송 이벤트용 dto 생성 * feat: 알림 전송 서비스에 이벤트 리스너 등록 * feat: 메시지 서비스, 알림 전송 이벤트 분리 * feat: 입찰 서비스, 알림 전송 이벤트 분리 * test: 알림 전송 이벤트 리스너 테스트 추가 * refactor: 알림 서비스와 알림 이벤트 리스너 분리 * test: 메시지 알림 이벤트 리스너 테스트 추가 * refactor: 리다이렉트와 이미지 주소 변환 로직 이벤트 리스너로 이동 * feat: 이벤트 발행 시 전달할 이벤트 dto 생성 * feat: 입찰 이벤트 생성 * test: dto 내부의 주소 변환 로직 이동으로 인해 깨지는 테스트 수정 * test: 이벤트 리스너에 입찰 테스트 추가되어 주석 삭제 * test: 픽스처 네이밍 변경 * refactor: 이벤트 퍼블리셔 네이밍 변경 * test: 알림 실패 시에도 메시지 커밋되는지 확인하는 테스트 추가 * refactor: 이벤트 전송을 위한 메서드 분리 * refactor: 길어지는 메서드 체이닝 변수로 추출 * fix: 머지 시 누락된 테스트 코드 변경 * refactor: 불필요한 필드 제거 * refactor: EventListener 트랜잭션 이벤트 리스너로 변경 * refactor: 트랜잭션 분리 제거, 예외를 처리하지 않고 던지도록 변경 * refactor: 알림 서비스에서 발생하는 예외 이벤트 리스너에서 처리하도록 수정 * test: 이벤트 리스너 알림 전송 실패 케이스 테스트 추가 * test: 이벤트 발행 테스트를 알림 서비스가 아닌 이벤트 리스너로 테스트하도록 변경 * test: 채팅방 로직 변경에 따라 추가가 필요했던 테스트 todo 삭제 * refactor: 관심사 분리를 위해 dto 네이밍 변경 * test: 이벤트 메서드 실행 시 알림 메서드 실행되는지 여부 확인하는 테스트 추가 * test: 결과에 해당하는 코드 then절로 이동
* feat: 사용자 평가 엔티티 추가 * feat: 사용자 평가 레포지토리 추가 * feat: 사용자 신뢰도 계산 기능 추가 * feat: 사용자 신뢰도 제출 및 조회 서비스 추가 * feat: 사용자 신뢰도 제출 및 조회 컨트롤러 추가 * refactor: 개행 및 메서드 분리를 통해 가독성 개선 * feat: 사용자 평가 관련 flyway 스크립트 추가 * docs: 유저 상호 평가 api 문서화 추가 * feat: 평가 등록 시 자격이 있는지 검증 로직 추가 * rename: request dto와 response dto 패키지 이동 * rename: 지정한 유저가 받은 모든 평가 목록을 가져오는 controller 메서드 이름 변경 * feat: 지정한 작성자가 지정한 경매에 제출한 평가를 조회하는 레포지토리 추가 * feat: 지정한 작성자가 지정한 경매에 제출한 평가를 조회하는 서비스 추가 * feat: 지정한 작성자가 지정한 경매에 제출한 평가를 조회하는 컨트롤러 추가 * docs: 사용자가 경매 거래 상대에게 작성한 평가 조회 부분 문서화 * test: 픽스처 객체 생성을 `@beforeEach`에서 하도록 변경 * test: 테스트에 Non-ASCII 경고 억제 어노테이션 추가 * refactor: null 데이터를 가진 dto를 매번 생성하지 않고 static 하게 갖고 있도록 변경 * refactor: 빌더를 사용한 생성자의 접근지정자를 private으로 변경 * style: 개행 수정 * feat: db에 평가 점수 필드에 null 불가 조건 추가 * test: 서비스 테스트에 픽스처 추가 * refactor: 사용하지 않는 `@EntityGraph` 제거 * feat: 지정한 평가 아이디로 평가를 조회할 수 있는 api 추가 * docs: 지정한 평가 아이디로 평가를 조회할 수 있는 api 문서화 추가 * feat: 평가 점수를 나타내는 VO 추가 * test: 픽스처 누락된 부분 추가 * feat: 사용자가 경매 거래에 작성한 평가 조회 uri 변경 * docs: 사용자가 경매 거래에 작성한 평가 조회 uri 변경에 따른 문서화 수정 * feat: 사용자 엔티티에 신뢰도 값객체 적용 * fix: 사용자 엔티티에 신뢰도 값객체 적용으로 인한 컴파일 에러 해결 * refactor: 사용자 엔티티 생성자 필드의 `@NotNull`을 `@NonNull`로 변경 * test: 실패하는 테스트 해결 * refactor: 초기 상태의 신뢰도를 나타내는 상수를 활용하도록 생성자 로직 수정 * style: 불필요한 개행 제거 * ci: 충돌 해결 * ci: 충돌 해결
* ci: 젠킨스 스크립트 수정 * ci: 젠킨스 스크립트 불필요한 명령어 삭제
kwonyj1022
approved these changes
Oct 6, 2023
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.
📄 작업 내용 요약
develop 브랜치에 develop-be 브랜치 최신화
🙋🏻 리뷰 시 주의 깊게 확인해야 하는 코드
📎 Issue 번호