-
Notifications
You must be signed in to change notification settings - Fork 0
Feat: 일정 생성 API #21
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
Feat: 일정 생성 API #21
Conversation
Test Coverage Report
|
choiseoji
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.
수고하셨습니다 ~!!
| private final EventServiceImpl eventService; | ||
|
|
||
| @PostMapping | ||
| @PreAuthorize("hasRole('MANAGER') or hasRole('LEADER')") |
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 경로로 권한 검사 안해줘도 되는거죠??
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.
맞아요~
✨ 연관된 이슈
📝 작업 내용 (주요 변경 사항)
💬 리뷰 요구사항
일정이랑 러닝캡 연관관계 OneToOne -> ManyToOne으로 변경
(ddl-auto 옵션 배포환경에 적용해야돼서 create로 변경했는데 다음 PR에서 update로 변경 부탁드려요~)