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

[Refactor] ReportType 값이 유효하지 않은 경우 검증 추가 #161

Conversation

jj0526
Copy link
Contributor

@jj0526 jj0526 commented Feb 3, 2025

개요

PR 유형

어떤 변경 사항이 있나요?

  • 새로운 기능 추가
  • 버그 수정
  • CSS 등 사용자 UI 디자인 변경
  • 코드에 영향을 주지 않는 변경사항(오타 수정, 탭 사이즈 변경, 변수명 변경)
  • 코드 리팩토링
  • 주석 추가 및 수정
  • 문서 수정
  • 테스트 추가, 테스트 리팩토링
  • 빌드 부분 혹은 패키지 매니저 수정
  • 파일 혹은 폴더명 수정
  • 파일 혹은 폴더 삭제

PR Checklist

PR이 다음 요구 사항을 충족하는지 확인하세요.

  • 커밋 메시지 컨벤션에 맞게 작성했습니다.
  • 변경 사항에 대한 테스트를 했습니다.(버그 수정/기능에 대한 테스트).

📣 To Reviewers

  • reportType을 받았을 때 모두 대문자로 바꾸고, 포함되지 않는 경우는 null로 처리하였습니다
  • 서버 장애 발생 JSON parse error: Cannot deserialize value of type com.example.eatmate.app.domain.report.domain.ReportType from String "욕설": not one of the values accepted for Enum class: [OTHER, OFFENSIVE, HARASSMENT] : 이 오류는 받은 값이 Enum에 포함되지 않아 생기는 오류로 서버 내부 에러로 처리되기에 수정하였습니다

@jj0526 jj0526 added refactor 코드 리팩토링 시현 labels Feb 3, 2025
@jj0526 jj0526 requested a review from a team February 3, 2025 07:47
@jj0526 jj0526 self-assigned this Feb 3, 2025
@jj0526 jj0526 linked an issue Feb 3, 2025 that may be closed by this pull request
1 task
@jj0526 jj0526 changed the title [Refactor] ReportType 값이 유효하지 않은 경우 예외 처리 추가 [Refactor] ReportType 값이 유효하지 않은 경우 검증 추가 Feb 3, 2025
Copy link
Contributor

@dyk-im dyk-im left a comment

Choose a reason for hiding this comment

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

리뷰 받으신 부분 외에는 문제가 없는 것 같습니다!

Copy link
Contributor

@seokjun01 seokjun01 left a comment

Choose a reason for hiding this comment

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

코드 확인하였습니다 고생하셨습니다.😄

@jj0526 jj0526 merged commit 8b08159 into dev Feb 4, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
refactor 코드 리팩토링 시현
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Refactor] ReportType 값이 유효하지 않은 경우 예외 처리 추가
4 participants