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

[BE] fix: Swagger 문서에 잘못된 Parameter 수정 및 형식 통일 (#959) #961

Merged
merged 4 commits into from
May 13, 2024

Conversation

seokjin8678
Copy link
Collaborator

📌 관련 이슈

✨ PR 세부 내용

이슈에 적은 내용대로 사용되지 않는 Pageable 파라미터가 표시되는 문제를 해결했습니다.

또한 각자 Controller를 작업하다 보니, Swagger 문서의 내용이 통일되지 않아 혼란이 있는 관계로 형식을 모두 통일 시켰습니다.

Swagger가 자동화를 해주는 것은 참 좋긴 한데.. 오히려 불편하기도 하네요. 😂

그렇다고 REST Docs를 사용하기에는 공수가 너무 많이 필요하기도 하고...

아무튼 그렇습니다. 😂

@seokjin8678 seokjin8678 added BE 백엔드에 관련된 작업 🛠 수정 수정에 관련된 작업 labels May 11, 2024
@seokjin8678 seokjin8678 self-assigned this May 11, 2024
@github-actions github-actions bot requested review from BGuga, carsago and xxeol2 May 11, 2024 09:03
Copy link

Test Results

226 files  226 suites   29s ⏱️
751 tests 751 ✅ 0 💤 0 ❌
770 runs  770 ✅ 0 💤 0 ❌

Results for commit 1c0902d.

@seokjin8678
Copy link
Collaborator Author

시간이 오래지났고, 수정에 관한 PR이기 때문에, 리뷰 기다리지 않고 머지하겠습니다!

@seokjin8678 seokjin8678 merged commit 6faa3a9 into dev May 13, 2024
3 checks passed
@seokjin8678 seokjin8678 deleted the feat/#959 branch May 13, 2024 03:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BE 백엔드에 관련된 작업 🛠 수정 수정에 관련된 작업
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BE] Swagger 문서에 사용되지 않는 Pageable 파라미터가 표시되는 문제를 해결한다.
1 participant