4주차 멘토링 코드리뷰용 PR #135
Open
4주차 멘토링 코드리뷰용 PR #135
Conversation
This file contains hidden or 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
* feat: 테스트코드 * feat: ProfileMetadata서비스 테스트코드 * fix: 주석제거
* refactor:service 공통로직 수정 * feat:쿠폰, 레슨 스케쥴러 추가 * feat:coupon, lesson 테스트코드추가 * refactor:lessonService 상수화변경 * feat:쿠폰삭제기능추가 * feat:couponServiceTest 삭제테스트추가 * fix:conflict 해결 * refactor:쿼리성능개선 * refactor:adminCouponController 검증로직 단순화 * feat:레슨생성시쿨타임제한추가
* feat: 누락된 검색 조건 및 필터링 추가 * fix: 누락된 생성 조건 추가 * fix: 누락된 생성 조건 추가
* refactor: 컨트롤러 코드 수정 * feat: dto -> record 클래스로 변경 * refactor: 서비스, 매퍼 수정 * refactor: dto -> record로 수정 및 그에 따른 서비스, 컨트롤러, 매퍼 변환 * refactor: record로 수정함에 따른 테스트 코드 수정 * feat: 서비스 분리 * feat: comment 관련 n+1 문제 -> 인터페이스 프로젝션 통해 해결 * feat: review n+1 문제 fetch join 통해 해결 * feat: comment 관련 테스트 코드 수정 * feat: 결제 관련 메서드 추가 * feat: mapper 분리 통해 코드 재구성 * feat: 결제 관련 N+1 문제 해결 -> fetch join * feat: refund 관련 공통 메서드 생성 * feat: 타임존 관련 추가
* refactor: email 추가 * refactor: 회원탈퇴 검증 추가 * refactor: 관리자 api , 페이지 검증
* refactor: email 추가 * refactor: 회원탈퇴 검증 추가 * refactor: 관리자 api , 페이지 검증 * feat : 메타데이터 스케줄링처리 * refactor: admin 열어두어 서비스단에서 검증 걸리도록 * refactor: 시큐리티 admin 검증 * refactor: 더미데이터 스케줄링에 맞게 변경 * refactor: 기존 메타데이터 업데이트 제거 * fix: 미사용 변수 제거
* refactor: 랭킹 집계 기준 변경 20 -> 5 * fix : 평점 소수점 2번째 자리까지
* refactor:lessonParticipant 로직 수정 * fix:checkStyle 수정 * refactor:테스트코드도 변경에 맞게 수정 * refactor: 메서드 분리
* feat: 쿠폰 관련 페이징 구조 변환 * feat: 레슨 관련 페이징 구조 변환 * Revert "feat: 레슨 관련 페이징 구조 변환" This reverts commit b80805c. * feat: review 레포 관련 쿼리 수정 * feat: 리뷰 작성 기간 로직 수정 * fix: 오타 수정
* feat: S3 업로드, 조회 로직 구현 * fix: 매핑 url 오탈자 수정
* feat: S3 업로드, 조회 로직 구현 * fix: 매핑 url 오탈자 수정 * feat: base response 형식 적용 * style: 스타일 미적용 수정
* feat: 컨트롤러 limit, page 요청 통일 * Feat: 쿼리 대문자로 변경 * refactor: 쿠폰 관련 필요없는 주석 제거 * feat: 레슨 관련 페이징 통합 및 구조 정리 * refactor: 쿼리 대소문자 수정 * feat: 프로필 관련 페이징 구조 통일 * feat: 리뷰 관련 n+1 문제 해결 -> profile 쪽 같이 join 통해 * feat: 레슨 관련 N + 1 문제 해결
* feat: 레슨 신청 결제 대기 상태 도입 * feat: 선착순 레슨 신청시 openTime 비교 예외처리 * fix: 불필요한 쿼리 검색 조건 삭제
* refactor: 쿼리 관련 리팩토링 * feat: 서비스 추가
* feat: 동시성 테스트를 위한 컨트롤러 작성 * feat: 동시성 테스트를 위한 DTO 작성 * feat: 동시성 테스트를 위한 유저 자동생성 로직 서비스 작성 * feat: 동시성 테스트를 위한 Spring SecurityFilterChain 수정
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.
4주차 멘토링 코드리뷰용 PR