플레이리스트 추가 기능 #150
Merged
플레이리스트 추가 기능 #150
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
* playlist entity 생성 * user와 관계 설정
* playlist 와 music 의 관계를 정리해주는 music_playlist 테이블 생성
* manyToOne, oneToMany 를 적극적으로 사용하여 설정 * musicId numnber 타입으로 수정
* playlist 요청 처리를 위한 모듈 생성
* playlist 를 생성할 때 사용하는 DTO 객체 추가
* POST /playlists 로 생성 가능
* id 의 타입 string -> number * playlistId -> playlist_id
* POST /playlists/{playlist_id} 로 요청
* 해당 플레이리스트가 있는지, 음악이 있는지 확인 -> 없으면 400 에러
* 추가가 됐으면 200 응답
* PlaylistService 를 주입하여 signup 을 한 후에 최근 재생 목록 플레이리스트를 바로 생성하도록 구현
Closed
* 테스트 통과되도록 모듈 의존관계 추가
* "최근 재생 목록" -> 상수로 처리
sk000801
reviewed
Nov 22, 2023
| @@ -49,6 +51,9 @@ export class AuthService { | |||
| }); | |||
| await this.userRepository.save(newUser); | |||
|
|
|||
| this.playlistService.createPlaylist(newUser.user_id, { | |||
There was a problem hiding this comment.
제가 단편적인 부분만 본 것 같긴 한데 로그인 할 때마다 유저에게 재생 목록을 새로 제공해 주는 건가요?!
There was a problem hiding this comment.
로그인이 아니라 새로운 회원이 가입했을 때 그 유저의 최근 재생 목록을 만들어주는 작업입니다!
sk000801
reviewed
Nov 22, 2023
| user_id: string; | ||
| user: User; | ||
|
|
||
| @OneToMany(() => Music_Playlist, (music_playlist) => music_playlist.music) |
There was a problem hiding this comment.
아까 알려주신 부분이 요 부분들이군요 유념하겠습니다 🙇♀️
sk000801
reviewed
Nov 22, 2023
| playlistId, | ||
| musicId, | ||
| ); | ||
| return { music_playlist_id: music_playlist_id }; |
There was a problem hiding this comment.
요거 return {music_playlist_id}로 두어도 같은 의미에용
sk000801
approved these changes
Nov 22, 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.
Issue
Overview
Playlist,Music_Playlistentity 생성@ManyToOne,@OneToMany를 활용해서 관계 추가, 개선POST/playlists플레이리스트 추가 APIPOST/playlists/{플레이리스트 ID}플레이리스트에 음악 추가 API-> 없으면 400 응답
최근 재생 목록플레이리스트 추가하도록 구현Screenshot
POST/playlists플레이리스트 추가POST/playlists/{playlistId}플레이리스트에 음악 추가최근 재생 목록플레이리스트 생성To Reviewers
entity 에서 관계설정 부분을 수정했습니다.
ManyToOne, OneToMany 를 사용해서 관계 설정하는 부분과 create, countBy, findBy 하는 부분에서 변경이 있으니 다음 구현부터 적용 부탁드립니다!