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 : BadRequestException 발생시 타입에러로 인한 오류 수정 #43

Merged
merged 3 commits into from
Jan 11, 2024

Conversation

jifrozen0110
Copy link
Contributor

@jifrozen0110 jifrozen0110 commented Jan 10, 2024

  • ✅ PR 제목의 형식을 잘 작성했나요? e.g. feat: PR을 등록한다.
  • ✅ 테스트는 잘 통과했나요?
  • ✅ 빌드는 성공했나요?
  • ✅ 불필요한 코드는 제거했나요?
  • ✅ 이슈는 등록했나요?
  • ✅ 라벨은 등록했나요?
  • ✅ 코드 컨벤션은 지켰나요?
  • ✅ pr 워크플로우를 돌려주세요.

작업 내용

BadRequestException 발생시 타입에러로 인한 BadRequestException을 내려주는것이 아닌
에러를 발생시켜 수정하였습니다.
#42 이슈 참고 부탁드립니다.

보미님께서 발견해주신 에러로 실제 배포는 이미 수정된 파일이 배포된것으로 확인되었습니다!

Google 관련 test 정상작동하는것 확인하였습니다.
원래 정상작동하였는데 제가 잘못본것같습니다. 죄송합니다!
주석 제거하겠습니다.

스크린샷

주의사항

Closes #42

BadRequestException 발생시
타입에러로 인해 오류가 생기는 부분을 수정하였습니다.
Google 테스트 코드 정상 작동 확인 후
주석 제거하였습니다.
Copy link

Quality Gate Passed Quality Gate passed

The SonarCloud Quality Gate passed, but some issues were introduced.

4 New issues
0 Security Hotspots
92.3% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

Copy link
Contributor

@soonhankwon soonhankwon left a comment

Choose a reason for hiding this comment

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

수고하셨습니다 👍

@jifrozen0110 jifrozen0110 merged commit 67adbdf into develop Jan 11, 2024
2 checks passed
@jifrozen0110 jifrozen0110 deleted the fix/issue-42 branch February 21, 2024 10:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BUG] BadRequestException 타입 불일치
2 participants