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

[#1571] TimePicker > 외부에서의 modelValue 값 변화를 감지하지 못하는 문제 수정 #1572

Merged
merged 2 commits into from
Jan 12, 2024

Conversation

chichoon
Copy link
Collaborator

@chichoon chichoon commented Jan 10, 2024

이슈

Jan-09-2024 14-29-24

  • 기존의 TimePicker 는 최초로 modelValue 를 받아왔을 때 단 한 번만 값 초기화를 수행하므로 이후에 modelValue 가 외부에서 변경되었을 때 이를 반영하지 못하는 문제가 있었습니다.
  • Edit Window 등에서 API 호출을 통해 데이터를 받아올 경우, Props 가 뒤늦게 변경되어 TimePicker에 해당 값이 반영이 되지 않아 초기값 설정이 불가합니다.
  • (change 버튼은 디버깅용으로, 외부에서 값을 수정하는 것을 테스트하는 용도입니다)

해결

Jan-10-2024 10-06-22

  • modelValue 를 computed 를 통해 연결하여 외부에서 값이 변경되어도 즉각 반영될 수 있게끔 하였습니다.
  • validator 조건문 오류를 수정하였습니다 (기존에는 startTime 만 검사)
  • time, startTime, endTime 을 time 단일 값으로 통일하였습니다

…m#1571)

#############################
 - modelValue 가 외부에서 변경되어도 컴포넌트 내부에서 이를 감지할 수 있도록 수정
@sonky740 sonky740 merged commit adda8a8 into ex-em:3.0 Jan 12, 2024
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.

2 participants