-
Notifications
You must be signed in to change notification settings - Fork 8
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
[AN/USER] API 변경 사항 반영 (#792) #795
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
SeongHoonC
approved these changes
Mar 19, 2024
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
re4rk
approved these changes
Mar 19, 2024
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lgtm!
BGuga
added a commit
that referenced
this pull request
Mar 29, 2024
* [BE] feat: 어드민 축제 상세 조회 기능 추가 (#780) (#782) * feat: 어드민 축제 상세 조회 Repository 추가 * chore: 테스트 코드 sout 제거 * feat: 어드민 축제 상세 조회 API 추가 * [BE] refactor: 어드민 패키지 구조 정리 (#783) (#785) * refactor: 어드민 패키지 정리 * refactor: Artist 도메인에 Admin 의존 제거 - Application 레이어에 Presentation 레이어 의존 제거 * test: ArtistCommandService 단위 테스트 추가 - 기존 ArtistCommandServiceIntegrationTest 대체 * refactor: ArtistV1Response -> AdminArtistV1Response 이름 변경 - 사용처에 따른 명확한 이름 사용 (추후 Artist 도메인에서 이름이 겹치는 것을 방지) * [AN/USER] API 변경 사항 반영 (#792) (#795) * fix: FestivalResponse 수정 * fix: ArtistResponse 수정 * [BE] feat: 컨트롤러의 핸들러 메서드에 어노테이션을 붙여 인증을 수행하는 기능 추가 (#789) (#790) * [BE] feat: 사용자의 FCM을 등록하는 API 분리 (#579) (#582) * refactor: AuthController FCM 의존 제거 * feat: 사용자의 FcmToken을 저장하는 API 추가 * refactor: MemberFCMService 로직 리팩터링 - 조회 메서드 `List<String>` 리턴 타입 변경 - 삭제 메서드 `@Async` 제거 * refactor: 메서드 명 명확하게 변경 * test: FCMNotificationEventListenerTest 테스트 수정 - 이벤트 수신 시 메서드 실행되는지 확인하도록 변경 * fix: 머지 충돌 해결 * feat: 회원 삭제 시 FCM 삭제 로직 구현 * fix: 로그인 요청에 FCM 토큰 제거 * refactor: Member 생성, 삭제 이벤트 명 변경, 필드 수정 * [BE] fix: AnnotationDelegateInterceptor에 경로 추가하여 ClassCastException 방지 (#804) (#805) * fix: AnnotationDelegateInterceptor 경로 추가하여 ClassCastException 방지 * fix: 테스트 컨트롤러에 /api 경로 추가 * [BE] refactor: 레거시 코드 제거 (#796) (#801) * refactor: AdminController 일부 디버깅 기능 제외하고 제거 * refactor: Festival, School, Stage 레거시 코드 제거 * test: 큐컴버 테스트 새로운 API 사용하도록 변경 * refactor: 일부 개선 대상에 대해 TODO 주석 추가 * refactor: StageQueryInfo, FestivalQueryInfo 사용하지 않는 메서드 삭제 * refactor: 사용하지 않는 에러코드 삭제 * [BE] refactor: 테스트 픽스쳐 개선 (#806) (#807) refactor: 테스트 픽스쳐 개선 - 형식 통일 - 패키지 분리 * feat: 아티스트 북마크 목록 조회 구현 * feat: 아티스트 북마크 저장 및 삭제 구현 * feat: 아티스트 북마크 컨트롤러 구현 * refactor: artist 존재 여부를 existsById 를 통해서 처리한다 * test: 메모리 repository 에 existsById 추가 * feat: 아티스트가 없을 때 예외를 발생시키도록 변경 * refactor: MemberRepository 가 Repository 를 상속받도록 변경 * test: MemoryMemberRepository 구현 및 테스트 적용 --------- Co-authored-by: Seokjin Jeon <[email protected]> Co-authored-by: 해시 <[email protected]>
seokjin8678
added a commit
that referenced
this pull request
Apr 3, 2024
* feat: Bookmark 도메인 생성 * fix: sql 오타 수정 * style: 줄바꿈 제거 * fix: bookmark_type 컬럼 길이 10자 수정 * feat: BookmarkRepository 유니크 인덱스 삭제 메서드 추가 및 Fake 구현체 추가 * fix: 누락된 public 접근자 추가 * [BE] feat: 아티스트 북마크 구현(#787) (#816) * [BE] feat: 어드민 축제 상세 조회 기능 추가 (#780) (#782) * feat: 어드민 축제 상세 조회 Repository 추가 * chore: 테스트 코드 sout 제거 * feat: 어드민 축제 상세 조회 API 추가 * [BE] refactor: 어드민 패키지 구조 정리 (#783) (#785) * refactor: 어드민 패키지 정리 * refactor: Artist 도메인에 Admin 의존 제거 - Application 레이어에 Presentation 레이어 의존 제거 * test: ArtistCommandService 단위 테스트 추가 - 기존 ArtistCommandServiceIntegrationTest 대체 * refactor: ArtistV1Response -> AdminArtistV1Response 이름 변경 - 사용처에 따른 명확한 이름 사용 (추후 Artist 도메인에서 이름이 겹치는 것을 방지) * [AN/USER] API 변경 사항 반영 (#792) (#795) * fix: FestivalResponse 수정 * fix: ArtistResponse 수정 * [BE] feat: 컨트롤러의 핸들러 메서드에 어노테이션을 붙여 인증을 수행하는 기능 추가 (#789) (#790) * [BE] feat: 사용자의 FCM을 등록하는 API 분리 (#579) (#582) * refactor: AuthController FCM 의존 제거 * feat: 사용자의 FcmToken을 저장하는 API 추가 * refactor: MemberFCMService 로직 리팩터링 - 조회 메서드 `List<String>` 리턴 타입 변경 - 삭제 메서드 `@Async` 제거 * refactor: 메서드 명 명확하게 변경 * test: FCMNotificationEventListenerTest 테스트 수정 - 이벤트 수신 시 메서드 실행되는지 확인하도록 변경 * fix: 머지 충돌 해결 * feat: 회원 삭제 시 FCM 삭제 로직 구현 * fix: 로그인 요청에 FCM 토큰 제거 * refactor: Member 생성, 삭제 이벤트 명 변경, 필드 수정 * [BE] fix: AnnotationDelegateInterceptor에 경로 추가하여 ClassCastException 방지 (#804) (#805) * fix: AnnotationDelegateInterceptor 경로 추가하여 ClassCastException 방지 * fix: 테스트 컨트롤러에 /api 경로 추가 * [BE] refactor: 레거시 코드 제거 (#796) (#801) * refactor: AdminController 일부 디버깅 기능 제외하고 제거 * refactor: Festival, School, Stage 레거시 코드 제거 * test: 큐컴버 테스트 새로운 API 사용하도록 변경 * refactor: 일부 개선 대상에 대해 TODO 주석 추가 * refactor: StageQueryInfo, FestivalQueryInfo 사용하지 않는 메서드 삭제 * refactor: 사용하지 않는 에러코드 삭제 * [BE] refactor: 테스트 픽스쳐 개선 (#806) (#807) refactor: 테스트 픽스쳐 개선 - 형식 통일 - 패키지 분리 * feat: 아티스트 북마크 목록 조회 구현 * feat: 아티스트 북마크 저장 및 삭제 구현 * feat: 아티스트 북마크 컨트롤러 구현 * refactor: artist 존재 여부를 existsById 를 통해서 처리한다 * test: 메모리 repository 에 existsById 추가 * feat: 아티스트가 없을 때 예외를 발생시키도록 변경 * refactor: MemberRepository 가 Repository 를 상속받도록 변경 * test: MemoryMemberRepository 구현 및 테스트 적용 --------- Co-authored-by: Seokjin Jeon <[email protected]> Co-authored-by: 해시 <[email protected]> * [BE] feat: 축제 북마크 기능 구현 (#788) (#791) * feat: FestivalBookmarkCommandService 추가 * feat: FestivalBookmarkV1QueryService 추가 * feat: FestivalBookmarkV1Controller 추가 * chore: 스웨거 어노테이션 추가 * feat: DTO 형식 클라이언트 요구사항 반영 * chore: TODO 제거 * chore: Swagger summary 추가 * fix: 머지 충돌 해결 * [BE] feat: 학교 북마크 기능 추가 (#787) (#809) * feat: 학교 북마크 저장 로직 추가 * refactor: 북마크 검증 Validator 클래스로 분리 * refactor: 북마크 최대 갯수 매직넘버 제거 * feat: 회원의 북마크 학교 목록을 조회하는 기능 추가 * refactor: 테스트 메소드명 변경 * refactor: 테스트 클래스명이 달랐던 문제 변경 * refactor: DTO 포장하여 전달 * feat: 학교 북마크 조회 API 추가 * feat: swagger 설정 추가 * refactor: 북마크 테스트시 회원 저장하도록 변경 * refactor: 학교 북마크 삭제 기능 추가 * fix: 학교 조회시 CustomException throw * feat: 학교 검색 DTO 가져오기 * refactor: 학교 검색 DTO 공용적으로 사용하도록 수정 * refactor: 북마크 조회 API 테스트 구체적으로 변경 * refactor: 통합 테스트 클래스명 컨벤션에 맞게 변경 * refactor: 조회 Service에 readonly 옵션 추가 * refactor: 사용하지 않는 클래스 제거 * refactor: SchoolBookmarkAppendValidator 로 클래스명 변경 * refactor: 테스트에서 repository 타입을 interface로 변경 * refactor: 컨벤션에 맞게 개행 및 클래스명 변경 * refactor: 회원 학교 북마크 조회 시 join절에 조건을 명시하도록 변경 * refactor: 북마크 예외 갯수 초과 에러 메시지 변경 * refactor: SchoolRepository를 Repository를 상속하도록 변경 * refactor: 학교 존재 검증 Validator에서 수행하도록 변경 * test: MemorySchoolRepositoryTest 추가 * refactor: merge 충돌 방지를 위해 사용하지 않는 클래스 제거 * refactor: 사용하지 않는 메소드 제거 * refactor: SchoolBookmarkAppendValidator 삭제 * refactor: MemorySchoolRepository 구현 변경 * fix: 테스트 깨지던 것 수정 * style: 개행 수정 --------- Co-authored-by: seokjin8678 <[email protected]> * fix: 머지 충돌 해결 * refactor: Controller 형식 통일, 인증 기능 추가 * feat: 북마크 등록/삭제 API 추가 * refactor: Bookmark 서비스 로직 통합 * test: 테스트 코드 통합, 픽스쳐 사용하도록 변경 * refactor: DTO 패키지 이동, SchoolBookmarkInfoV1Response 추가 * refactor: QueryDslRepository 패키지 위치 이동 * fix: Controller 메서드 명 수정 * refactor: SchoolBookmarkV1Response의 생성시간 필드명을 createdAt으로 변경 * chore: artist response 필드명 변경 --------- Co-authored-by: BGuga <[email protected]> Co-authored-by: 해시 <[email protected]> Co-authored-by: Hyun-Seo Oh / 오현서 <[email protected]> Co-authored-by: carsago <[email protected]>
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.
📌 관련 이슈
✨ PR 세부 내용
FestivalResponse, ArtistResponse 수정해서 서버 응답 정상적으로 받을 수 있도록 했습니다.