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.

Sign up for GitHub

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

Jump to bottom

[FEAT] 플레이리스트 API #60

Merged
merged 47 commits into from
Sep 26, 2024
Merged

[FEAT] 플레이리스트 API #60

merged 47 commits into from
Sep 26, 2024

Conversation

dgh06175
Copy link
Contributor

@dgh06175 dgh06175 commented Sep 21, 2024 •
edited
Loading

PR 타입(하나 이상의 PR 타입을 선택해주세요)

  • 기능 추가
  • 기능 삭제
  • 버그 수정
  • 의존성, 환경 변수, 빌드 관련 코드 업데이트

반영 브랜치

feat/#44/playlist -> develop

변경 사항

플레이리스트 관련 API를 추가했습니다.

테스트 결과

image

@dgh06175
feat: #44 기본 클래스 생성
8414fd4
@dgh06175
feat: #44 기본 클래스 생성 및 스프링 빈 연결
6e24f69
@dgh06175
feat: #44 1차 컨트롤러, 서비스, dto 구현
f0981c7
@dgh06175
feat: #44 2차 컨트롤러, 서비스, dto 구현
712b992
@dgh06175
refactor: #44 매개변수 수정
420f6e3
@dgh06175
feat: #44 플레이리스트 엔티티에 업로더를 의미하는 멤버 필드 추가
e3dd257
@dgh06175
test: 트랙 테스트 메소드 이름을 서비스 메소드 네이밍과 동일하게 직관적으로 변경
7625481
@dgh06175
feat: #44 플레이리스트 안의 트랙 정보를 가지고 있는 PlaylistTrack 엔티티의 레파지토리 추가
ffa1b22
@dgh06175
test: #44 실패하는 테스트 작성
bf8c193
@dgh06175
test: #44 테스트 클래스 네이밍 오타 수정, TrackServiceTest 일부 메소드 재귀 값 비교로 수정
fa680fb
@dgh06175
test: #44 테스트 코드 중복 코드 제거
c5eda3f
@dgh06175
feat: #44 PlaylistTrack BaseEntity 상속
00d0bdf
@dgh06175
feat: #44 플레이리스트 생성
80825e8
@dgh06175
feat: #44 플레이리스트 목록 가져오기
c1b35a6
@dgh06175
feat: #44 플레이리스트 목록 검색
1f39930
@dgh06175
test: #44 플레이리스트 임시 데이터 추가 및 페이징, 검색 등 테스트 세분화하여 추가
c8872ca
@dgh06175
feat: #44 플레이리스트 에러코드 작성
59fffee
@dgh06175
feat: 멤버 아이디와 clientId 가 동일하면 동일하다고 취급
5f37e2b
@dgh06175
feat: 트랙 삭제시 비교 로직에서 멤버 동일한지 비교할떄 id 비교가 아닌 Member 객체 비교로 변환
38b2a36
@dgh06175
feat: #44 플레이리스트 에러 코드 작성
6cbae50
@dgh06175
test: #44 삭제 테스트 작성
0167ac7
@dgh06175
feat: #44 플레이리스트 삭제
d3331fb
@dgh06175
feat: #44 플레이리스트 디테일 조회
7970e3a
@dgh06175
test: #44 테스트 잘못 작성한것 수정
67bf465
@dgh06175
feat: #44 플레이리스트에 트랙 추가
c11c131
@dgh06175
test: 테스트 데이터 추가에 따른 테스트 코드 일부 수정:
3fb064c
@dgh06175
chore: #44 playlist 엔드포인트 접근 허용
022fcbf
@dgh06175
feat: #44 어노테이션 추가
072a7db
@dgh06175 dgh06175 added the enhancement New feature or request label Sep 21, 2024
@dgh06175 dgh06175 requested a review from kyxxgsoo September 21, 2024 07:42
@dgh06175 dgh06175 requested a review from Younkyum September 21, 2024 07:42
@dgh06175 dgh06175 self-assigned this Sep 21, 2024
@dgh06175 dgh06175 linked an issue Sep 21, 2024 that may be closed by this pull request
[FEAT] 플레이리스트 API #44
Closed
2 tasks
@dgh06175
docs: #44 파라미터 기본값 및 설명 추가
6a25b6b
@dgh06175
docs: #44 파라미터 기본값 및 설명 추가
8064806
@dgh06175
feat: #44 플레이리스트 제목과 이미지 수정 API
26c05d7
@dgh06175
feat: #44 플레이리스트에 트랙 추가 시 중복 트랙 검사
57a7906
@dgh06175
feat: #44 플레이리스트 제목 및 이미지 수정
ffe8863
@dgh06175
feat: #44 플레이리스트 트랙 순서 변경, 플레이리스트 트랙 삭제 API 컨트롤러 작성
7bb7774
@dgh06175
test: #44 빈 플레이리스트 테스트 케이스 추가 및 기존 테스트 코드 알맞게 수정
05a2bae
@dgh06175
test: #44 플레이리스트에 트랙 추가 테스트
9f833f0
@dgh06175
test: #44 플레이리스트 트랙 순서 변경 테스트 작성
26d5a9b
@dgh06175
test: #44 플레이리스트 트랙 삭제 테스트 작성
5e0728f
@dgh06175
refactor: #44 플레이리스트 권한 검증 메소드 추출
d0d8453
@dgh06175
feat: #44 플레이리스트 엔티티 playlistTracks List -> Set 으로 변경
9b959b1
@dgh06175
feat: #44 플레이리스트 업데이트
a5f7aef
@dgh06175
feat: #44 플레이리스트 삭제
76b29ba
@dgh06175
feat: #44 에러 메세지 정리
0746c45
@dgh06175
refactor: #44 검증 메소드 추출 및 정리
f594a8a
@dgh06175
refactor: 공백 수정
1b9ac94
@dgh06175
feat: #44 트랙 정렬해서 반환하기
d5ac4ca
@dgh06175
feat: #44 플레이리스트 제목 검색 기능 성능 개선
3f27c9f
@dgh06175 dgh06175 requested review from kyxxgsoo and removed request for kyxxgsoo and Younkyum September 26, 2024 13:11
kyxxgsoo
kyxxgsoo approved these changes Sep 26, 2024
View reviewed changes
Copy link
Contributor

@kyxxgsoo kyxxgsoo left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@kyxxgsoo kyxxgsoo merged commit 35de1ce into develop Sep 26, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kyxxgsoo kyxxgsoo kyxxgsoo approved these changes

Assignees

@dgh06175 dgh06175

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[FEAT] 플레이리스트 API
2 participants
@dgh06175 @kyxxgsoo