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.

Sign up for GitHub

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

Jump to bottom

[BE] 요청에 대한 검증은 spring bean validation 을 사용해서 dto 에서 수행하게 한다. #572

Closed
nayonsoso opened this issue Sep 5, 2024 · 0 comments · Fixed by #958
Closed

[BE] 요청에 대한 검증은 spring bean validation 을 사용해서 dto 에서 수행하게 한다. #572

nayonsoso opened this issue Sep 5, 2024 · 0 comments · Fixed by #958
Assignees
@nayonsoso
Labels
✨ BE 🛠️ 리팩터링
Milestone
5차 스프린트

Comments

@nayonsoso
Copy link
Contributor

nayonsoso commented Sep 5, 2024 •
edited
Loading

🔍 설명

  • 테드가 한 서비스 리팩터링 PR 코드리뷰를 하며 나왔던 이야기 입니다.
  • 커비의 의견에 의하면, 체크박스 질문에 서술형 응답이 오는 것 & 서술형 질문에 체크박스 응답이 오는 것에 대한 검증이 '요청'에 대한 검증이므로, Mapper 에서 해주기도 애매해고, 객체에 대한 검증을 수행하는 Validator 에서 해주기도 애매했습니다.
  • 그래서 RequestValidator 라는 새로운 모듈을 만들어야 하나 생각을 했었지만, 팀원들의 논의 하에, 커스텀 Valid 를 만드는 것으로 결론이 나게 되었습니다.

🔥 할 일

  • Spring bean validator 를 사용해서 커스텀 요청 검증 로직을 작성합니다.

⏰ 예상 시간

  • 1시간 30분

🐴 할 말
@nayonsoso nayonsoso added this to the 5차 스프린트 milestone Sep 5, 2024
@nayonsoso nayonsoso self-assigned this Sep 23, 2024
@nayonsoso nayonsoso mentioned this issue Sep 23, 2024
Closed
@nayonsoso nayonsoso linked a pull request Sep 23, 2024 that will close this issue
[BE] feat: 리뷰 등록 요청 dto 형식 검증 로직을 SpringBeanValidator 를 통해 구현 #668
Closed
@nayonsoso nayonsoso mentioned this issue Nov 11, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@nayonsoso nayonsoso

Labels
✨ BE 🛠️ 리팩터링
Projects
2024-review-me
Status: Done
Milestone
5차 스프린트
1 participant
@nayonsoso