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

validation ,error 처리, 사용됨처리, 환불시 사용안됨처리 #20

Merged
merged 6 commits into from
Jul 17, 2024

Conversation

kihoo-ni
Copy link
Contributor

연관 이슈

이슈 번호 :

해당 PR은 어떤 유형의 PR인가요?

  • Bugfix
  • Feature
  • Rename
  • Test
  • Refactoring
  • Documentation content changes
  • Other... Please describe:

해당 기능에 대해 간략하게 설명해주세요

  • validation ,error 처리, 사용됨처리, 환불시 사용안됨처리 만들어놓기만 함 / front에는 아직 적용안함

해당 기능은 중대한 변경사항이 있나요?

  • Yes
  • No

결과

Copy link
Member

@taehwan01 taehwan01 left a comment

Choose a reason for hiding this comment

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

예외랑 유효성 처리하느라 모든 파일 검토하셨을텐데 고생하셨습니다

.append(", 입력된 값: ")
.append(fieldError.getRejectedValue())
.append("; ");
}
Copy link
Member

Choose a reason for hiding this comment

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

이렇게 처리하는 이유는 프론트 서버에서 에러 디코딩 하기 위함인가요?
API 서버에도 적용해볼까 해서 여쭙니당 👍

Copy link
Contributor Author

Choose a reason for hiding this comment

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

저의 경우 front server는 front 에서 back server 는 back에서 따로 error validation 처리를 해야한다고 생각해서
해당 validation error를 couponserver에서 잡아서 에러처리 하기위함으로 했습니다.
front에서도 validation 처리를해서 따로 만들어서 exceptionhandler로 처리해서 front validation은 따로 처리해야할 것 같습니다. error decoding은 https://velog.io/@hanqyu/Spring-OpenFeign-%EC%97%90%EB%9F%AC%EC%97%90-%EB%8D%94-%EB%A7%8E%EC%9D%80-%EC%A0%95%EB%B3%B4-%EB%8B%B4%EA%B8%B0-ErrorDecoder 요거 얘기하시는건가요 ?
fegin으로 오는것을 어떻게 분기처리할지 말씀하시는 거 같은데 아직 그것까지 생각하지 못했습니다.

@2897robo 2897robo merged commit ad9efcd into develop Jul 17, 2024
1 check passed
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.

3 participants