-
Notifications
You must be signed in to change notification settings - Fork 0
홈화면 고도화 #77
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
Conversation
Dev<->main 동기화
Dev <-> main
Dev<->main
Dev<->main 동기화
- Note 생성시 type 필드 포함 강제 - Note 조회시 type 필드 추가
…사 첨부되어 있지 않은 경우 에러 일으키도록
elive7
left a comment
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.
너무 늦게 리뷰해서 죄송해요... 수고하셨습니다
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.
테스트 코드까지 작성해주셨군요... 좋아요
| } | ||
|
|
||
| @GetMapping | ||
| @GetMapping("/users/notes") |
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.
/user로 시작하는데도 NoteController에 적은 이유가 따로 있나요? 수정을 할 필요는 없을 것 같은데 궁금해요
🍃 Pull Requests
⛳️ 작업한 브랜치
👷 작업한 내용
notes에 enum 타입의 컬럼type을 추가해야 한다.notes레코드는 모두FREE타입으로 처리한다.type을 추가해야 한다.type을 받는다.type에 따라 가사 첨부를 강제한다.LYRICS_ANALYSISFREE,QUESTION/api/v1/notes)의 URI를/api/v1/users/notes로 이전한다./api/v1/notes/favorite-artists)를 제거하고, 새로운 검색 API에 통합한다.findAllByArtistIds()에서ByArtistIds가 빠진 버전으로findAll()을 구현한다.getNotes()을 구현한다.NoteController.getNotes()에서hasFavoriteArtist파라미터에 따라서getNotes()/getNotesOfFavoriteArtists분기 처리한다.🚨 참고 사항