Skip to content
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

[#31] Swaager 추가 #33

Merged
merged 3 commits into from
Sep 4, 2024
Merged

[#31] Swaager 추가 #33

merged 3 commits into from
Sep 4, 2024

Conversation

youngreal
Copy link
Member

Changes 📝

컨트롤러 코드에 스웨거 코드가 침투되지않게 Interface 사용, 앞으로도 스웨거를 붙힐때 이와같이 사용하면 좋을것 같습니다.

* 컨트롤러 코드에서라도 최소한 Swagger 관련 어노테이션들을 걷어내기 인터페이스 사용

* JWT 토큰을 보내야하는 API에 공통으로 사용되는 스웨거 어노테이션들을 @AuthenticationCommonResponse로 응집
* 클라이언트가 보내주는 유저 색상과 authorization code에 대한 검증
@youngreal youngreal added the enhancement New feature or request label Sep 3, 2024
@youngreal youngreal self-assigned this Sep 3, 2024
@youngreal youngreal linked an issue Sep 3, 2024 that may be closed by this pull request
Copy link
Member

@min-0 min-0 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

수고하셨습니다 ~
머지된 이후에 회원 탈퇴 api도 swagger 작성해볼게요 !
코드로만 보기엔 아직 어떻게 잘 작성해야 할지 감이 안오네요 ㅎㅎ 꼭 알아야 할 점이 있다면 알려주세요 ~!

@youngreal
Copy link
Member Author

수고하셨습니다 ~ 머지된 이후에 회원 탈퇴 api도 swagger 작성해볼게요 ! 코드로만 보기엔 아직 어떻게 잘 작성해야 할지 감이 안오네요 ㅎㅎ 꼭 알아야 할 점이 있다면 알려주세요 ~!

회원탈퇴 API부터 한번 차근차근 달아보시고 궁금하신거 저한테 즉각 말씀주시면서 적응하시면 아마 금방하실거에요

@youngreal youngreal merged commit 8c24d83 into main Sep 4, 2024
1 check passed
@youngreal youngreal deleted the feat/#31 branch September 10, 2024 18:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Swagger 작성
2 participants