-
Notifications
You must be signed in to change notification settings - Fork 1
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
refactor: 어드민 유저 리팩토링 #1083
refactor: 어드민 유저 리팩토링 #1083
Conversation
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.
고생하셨습니다,,
충돌이 쉽지 않은 거였네요.. 이렇게 또 배워갑니다 ㅠㅠ,,,
@ParameterObject @ModelAttribute OwnersCondition ownersCondition, | ||
@Auth(permit = {ADMIN}) Integer adminId | ||
); | ||
} |
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.
고생하셨어요~ 제가 늦었네용 ㅎ..
🔥 연관 이슈
🚀 작업 내용
유저패키지를 유저, 사장, 학생 패키지로 나눴습니다.
유저패키지에 검증 클래스를 두었습니다.
읽기 힘든 로직을 개선했습니다.
💬 리뷰 중점사항