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] 제품/세차장 리뷰 도메인에 비속어 필터링 적용 및 글로벌 Exception 수정 #304

Merged

Conversation

linglong67
Copy link
Collaborator

@linglong67 linglong67 commented Mar 15, 2024

💡 Motivation and Context

1) 제품/세차장 리뷰 도메인에 비속어 필터링 적용
2) 관련 글로벌 Exception 수정


🔨 Modified

비속어 필터링 일부 내용 수정 및 도메인에 적용

  • 비속어 필터링 글로벌 익셉션 부분 수정
  • 세차장 리뷰 도메인에 추가 적용
  • ValidationMessage 파일 추가

🌟 More

  • flyway sql 모듈 이동
  • 개발-API, 배치 / 운영-배치 모듈 이미지 ecr로 올리기 위한 작업
  • 파일 확장자 검증 로직 추가


📋 커밋 전 체크리스트

  • 추가/변경에 대한 단위 테스트를 완료하였습니다.
  • 컨벤션에 맞게 작성하였습니다.

🤟🏻 PR로 완료된 이슈

closes #302
closes #303

@linglong67 linglong67 added 🐞 Bug 뭔가... 뭔가 잘못됐음... 🖥️ BackEnd 서버 관련 🔨 Refactor 이 코드는 아주 약간 더 클린코드에 가까워졌습니다... labels Mar 15, 2024
@linglong67 linglong67 self-assigned this Mar 15, 2024
String title,
@BadWordFilter
String contents,
@BadWordFilter(message = ValidationMessage.INVALID_WORD_PARAMETER) String title,
Copy link
Collaborator

Choose a reason for hiding this comment

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

필터 적용 확인하였습니다

Copy link
Collaborator

@chan99k chan99k left a comment

Choose a reason for hiding this comment

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

LGTM 변경사항 확인하였습니다

@@ -47,7 +48,7 @@ public ResponseEntity<ApiResponse<WashzoneReviewResponseDto>> getWashzoneReview(

@PostMapping
public <T> ResponseEntity<ApiResponse<T>> createWashzoneReview(
@RequestPart WashzoneReviewRequestDto washzoneReview,
@Valid @RequestPart WashzoneReviewRequestDto washzoneReview,
Copy link
Collaborator

Choose a reason for hiding this comment

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

👍🏻

Copy link
Collaborator

@HyunJunSon HyunJunSon left a comment

Choose a reason for hiding this comment

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

LGTM

@linglong67 linglong67 merged commit 6e55dad into Kernel360:develop Mar 15, 2024
@linglong67 linglong67 deleted the feature/fix-badword-exceoption branch March 15, 2024 09:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🖥️ BackEnd 서버 관련 🐞 Bug 뭔가... 뭔가 잘못됐음... 🔨 Refactor 이 코드는 아주 약간 더 클린코드에 가까워졌습니다...
Projects
None yet
Development

Successfully merging this pull request may close these issues.

비속어 필터링 Exception 관련 수정 세차장 리뷰에 비속어 필터링 적용
4 participants