-
Notifications
You must be signed in to change notification settings - Fork 0
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
도서 추천 api #105
도서 추천 api #105
Conversation
|
||
@NotBlank | ||
private String author; | ||
} |
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.
제가 알기론 숫자가 들어가면 @NotBlank
가 적합하지 않는 걸로 아는데 맞나요?
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.
문자열이라 상관없는거 같아요
} | ||
|
||
recRepository.save(recConverter.toEntity(writeRecRequest, type)); | ||
} | ||
} |
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.
저희 코드 스타일이 컨버터로 변환하고 그걸 객체를 따로 만들어서 save하는 느낌인걸로 알고 있어요
코드를 일관성있게 하면 좋을 거 같아요
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.
01ac3a2
수정했습니다
@@ -81,6 +81,9 @@ public SecurityFilterChain filterChain(HttpSecurity http) throws Exception { | |||
// rank | |||
.requestMatchers(HttpMethod.GET, "/api/v2/rank").authenticated() | |||
|
|||
// recommend | |||
.requestMatchers(HttpMethod.POST, "/api/v2/recommend").hasAnyAuthority(Authority.ROLE_TEACHER.name(), Authority.ROLE_STUDENT.name()) | |||
|
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.
e36184f
수정했습니다
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.
수고하셨어요
💡 개요
도서 추천 api를 구현했습니다
📃 작업내용
도서 추천 api를 구현했습니다
novel / essay 두 개의 타입을 가지고 있고
선생님 또는 도서부만 작성할 수 있습니다
🔀 변경사항
🙋♂️ 질문사항
⚗️ 사용법
🎸 기타