Skip to content
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
merged 2 commits into from
Mar 20, 2024
Merged

[AN/USER] API 변경 사항 반영 (#792) #795

merged 2 commits into from
Mar 20, 2024

Conversation

EmilyCh0
Copy link
Member

📌 관련 이슈

✨ PR 세부 내용

FestivalResponse, ArtistResponse 수정해서 서버 응답 정상적으로 받을 수 있도록 했습니다.

@EmilyCh0 EmilyCh0 added AN 안드로이드에 관련된 작업 USER labels Mar 18, 2024
@EmilyCh0 EmilyCh0 self-assigned this Mar 18, 2024
@github-actions github-actions bot requested review from re4rk and SeongHoonC March 18, 2024 11:48
Copy link
Member

@SeongHoonC SeongHoonC left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

Copy link
Collaborator

@re4rk re4rk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lgtm!

@EmilyCh0 EmilyCh0 merged commit 6f1de1b into dev Mar 20, 2024
4 checks passed
@EmilyCh0 EmilyCh0 deleted the feat/#792 branch March 20, 2024 11:20
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
Labels
AN 안드로이드에 관련된 작업 USER
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

[AN] API 변경 사항을 반영한다
3 participants