[IDLE-180] 센터프로필 조회및 수정 #20
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
monibu1548
reviewed
Jul 21, 2024
|
@monibu1548 넵~! 센터오시는날 찾아뵈도 될까요? 온라인도 좋습니다!! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
변경된점
ViewModel의 Output 프로퍼티의 타입을 Driver로 사용하여 Output에 대한 ViewModel - View간 통신을 단방향 통식으로 설정하였습니다.
궁금한점
Input 옵저버블로 전달받는 데이터를 서버로 전송하는 버튼이 있을 때 해당 구현을 하는 패턴이 궁금합니다.
이번 구현에서는 센터로부터 3가지 입력을 가져옵니다. (센터 전화번호, 센터소개, 센터 사진)
그리고 '저장' 버튼이 눌렸을 때, 입력받던 데이터들을 서버로 전송하기 위해 BehaviorRelay를 사용하였습니다.
버튼이 눌리는 경우 위 사진의 프로퍼티들로 부터 value프로퍼티를 호출하고 해당 값을 서버로 전송합니다.
더 나은 방식이 있는지 궁금합니다!