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

시간 validate가 내가 선택한 시간을 반영하지 않는 문제 수정 #539

Merged
merged 1 commit into from
Sep 12, 2024

Conversation

jaeml06
Copy link
Contributor

@jaeml06 jaeml06 commented Sep 9, 2024

PR의 목적이 무엇인가요?

현재 모임 수정하기 페이지에서 날짜가 언제인든 시간이 현재 날짜 기준으로
validate가 되어서 시간 설정에 문제가 발생
이를 해결하기위해 시간 validate에 Date도 props로 넘겨 받음

이슈 ID는 무엇인가요?

설명

시간만 설정일 경우 에러가 발생하지 않음
image

이전 시간을 선택 시, 에러가 발생함
image

질문 혹은 공유 사항 (Optional)

LabeledInput에 대한 시간에 대한 validateFun이 날짜에 의존하게 되어 구현이 어려웠습니다. 일단 해결하기는 하였으나 코드가 유지보수성이 떨어진다고 생각이 듭니다. 이를 어떻게 해결할 수 있을지 고민입니다. 좋은 아이디어가 있을까요?

@jaeml06 jaeml06 added FE 프론트엔드 관련 이슈입니다. 🚑 버그 fix (develop에서 파생된 문제) labels Sep 9, 2024
@jaeml06 jaeml06 requested review from ss0526100 and cys4585 September 9, 2024 10:07
@jaeml06 jaeml06 self-assigned this Sep 9, 2024
Copy link
Contributor

@cys4585 cys4585 left a comment

Choose a reason for hiding this comment

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

validation 함수를 Input 컴포넌트에 내려주지 않고, 상단부에서 처리해서 검사 결과를 내려주는 게 어떨까요?

이 유효성 검사 처리도 인풋 훅에서 하도록 하면 시간, 날짜 관련 유지보수성 부분에서도 괜찮을 것 같습니다!

@jaeml06
Copy link
Contributor Author

jaeml06 commented Sep 9, 2024

validation 함수를 Input 컴포넌트에 내려주지 않고, 상단부에서 처리해서 검사 결과를 내려주는 게 어떨까요?

이 유효성 검사 처리도 인풋 훅에서 하도록 하면 시간, 날짜 관련 유지보수성 부분에서도 괜찮을 것 같습니다!

그 방법도 고민하긴 했는데 급하게 작성하다보니 구조가 바로 안떠올라서 그냥 그대로 진행했네요

@ss0526100 ss0526100 merged commit 0fb3cd2 into develop-frontend Sep 12, 2024
1 check passed
@ay-eonii ay-eonii deleted the fix/#537 branch September 13, 2024 01:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
FE 프론트엔드 관련 이슈입니다. 🚑 버그 fix (develop에서 파생된 문제)
Projects
Status: 완료
Development

Successfully merging this pull request may close these issues.

3 participants