-
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
✨ Member 모듈 및 인증 AOP 구현 #14
The head ref may contain hidden characters: "13-Member-\uBAA8\uB4C8-\uBC0F-\uC778\uC99D-AOP-\uAD6C\uD604"
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.
휴무일에도 대단하시네요... 고생하셨습니다!
당장 생각나는 거로는 추후에 커뮤니티 관련 기능에서 사용자끼리 서로 정보를 볼때 기본 정보만 볼 수 있게 분리해놓으면 좋을거 같긴한데 성능이 차이가 많이날지는 모르겠네요
근데 DB 가 서로 달라서 추후 관리측면을 생각한다면 엔드포인트를 분리하는것도 좋아보입니다 !
맞습니다! 다음주에 한번 커뮤니티 기능적인 부분들이랑 어떤 필드들을 넣어야할지 픽스하는 시간 한번 가지면 좋을 것 같습니다! |
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로 회원의 기본 정보와 추가정보를 한번에 조회하는 기능을 만들었는데, 필요할시 회원 기본 정보 조회 엔드포인트 및 추가정보 조회 엔드포인트를 따로 만들어야할 것 같습니다.