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

[CHORE] Swagger 문서화 환경 세팅 #15

Merged
merged 6 commits into from
Dec 13, 2024
Merged

[CHORE] Swagger 문서화 환경 세팅 #15

merged 6 commits into from
Dec 13, 2024

Conversation

coli-geonwoo
Copy link
Contributor

@coli-geonwoo coli-geonwoo commented Dec 11, 2024

🚩 연관 이슈

closed #14

🗣️ 리뷰 요구사항 (선택)

Swagger 관련 환경 설정 완료하였습니다.

앞으로 문서화 관련 step과 Swagger를 처음 써보는 팀원을 위한 Swagger 코드 설명을 문서화해놓았습니다.

+) 에러 객체로 ProblemDetail을 반환토록 해놓았는데 괜찮은지 여쭈어요

@coli-geonwoo coli-geonwoo linked an issue Dec 11, 2024 that may be closed by this pull request
@unifolio0 unifolio0 added the chore 기타 label Dec 11, 2024
Copy link
Contributor

@leegwichan leegwichan left a comment

Choose a reason for hiding this comment

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

/noti

에러 객체로 ProblemDetail을 반환토록 해놓았는데 괜찮은지 여쭈어요

네, 근데 프론트랑 이야기 해보는게 좋아보이긴 합니다. (이것도 나중에 종강하고 나서 에러코드 관련한 이야기 해봐야겠네요. 클라에게 주는 정보 최소화 해야한다는 의견도 있으니까요.)

추가로 notion BE에 새로운 탭을 만들어서, 개인이 공유하고 싶은 내용을 정리해 놓는 것은 어떨까요? (저희 조는 "블로그" 탭이 있어, 따로 관리했습니다!)

Copy link
Contributor

@unifolio0 unifolio0 left a comment

Choose a reason for hiding this comment

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

/noti
approve합니다!

+) 에러 객체로 ProblemDetail을 반환토록 해놓았는데 괜찮은지 여쭈어요

저도 이건 프론트와 협의할 부분 같아요!

@coli-geonwoo
Copy link
Contributor Author

/noti

확인했습니다! 에러 객체 반환과 관련해서는 프론트 측에게 한번 물어볼게요!

@coli-geonwoo coli-geonwoo merged commit efda2eb into develop Dec 13, 2024
2 checks passed
@coli-geonwoo coli-geonwoo deleted the chore/#14 branch December 13, 2024 11:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore 기타
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

[CHORE] Swagger 문서화 환경 세팅
3 participants