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

[fix] 지원자 목록 보기 swagger 에러 수정 #179

Merged
merged 5 commits into from
Aug 4, 2024

Conversation

letskuku
Copy link
Member

@letskuku letskuku commented Jul 28, 2024

💡 [fix] 지원자 목록 보기 swagger 에러 수정

📝 Description


  • 지원자 목록 보기에 해당하는 api를 swagger에서 실행할 경우, 특정 조건들에서 에러가 발생하고 있어 관련 설정을 추가하였습니다.

    • required false인 파라미터들에 대해 값이 들어오지 않을 경우, 길이가 0인 문자열이 들어가도록 default value 설정
    • enum 타입인 파라미터들에 대해 값이 한글로 들어가고 있어 영어로 들어가도록 allowable values 설정
    • 설정 추가라 feat 커밋이라고 생각했는데 pr 올리려고보니 fix가 더 맞는 것 같네요...ㅠㅠㅠ
  • 수정한 부분들에 대한 테스트 코드를 작성했습니다.

    • 400 bad request가 뜨던 부분들을 고친거라 수정한 코드들이 200 ok가 나오는지를 위주로 테스트했습니다.

+) 인텔리제이 오류가 있어 dev 브랜치에 바로 푸시하는 이슈가 있었습니다 다음에 더 주의하도록 하겠습니다... dev 브랜치 코드는 되돌려놓았습니다...

🌲 Working Branch


feat/getApplications

📖 Related Issues


Copy link
Member

@yoonsseo yoonsseo left a comment

Choose a reason for hiding this comment

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

이런 이슈가 있었군용 수고하셨습니다~

@letskuku letskuku merged commit 907b10f into dev Aug 4, 2024
@letskuku letskuku deleted the feat/getApplications branch August 4, 2024 04:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants