Feature/circle detail screen #10
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
ItemClickListener 수정 ItemListenerInitializer 작성
CircleDetailViewModelFactory 추가 CircleDetailViewModel currentTabPosition 및 setTabPosition 추가
동아리 상세 화면 기존 requireActivity().supportFragmentManager 에서 childFragmentManager 로 전환 내부 tab 데이터는 기존 fragment 생성자 활용 방식에서 argument 활용 방식으로 전환
post 상세 화면 onCreateAnimator() 추가
뷰모델 내 데이터 로딩 작업과 uiState 갱신 작업 분리 refresh() 에서는 commentLoadingJob.invokeOnCompletion 을 통해 ui 갱신
postDetail 뷰모델 내 MutableLiveData.postValueWhenAnimFinished 작성 anim 이 종료된 경우에만 postValue, 그렇지 않은 경우 tmpState 에 저장 및 anim 종료 이후 postValue
기존 댓글 데이터 호출 후 본문 내용과 댓글을 한 번에 로딩하는 방식에서 본문 먼저 로딩 후 댓글은 호출되는대로 로딩하는 방식으로 수정
뷰모델에서 작성된 댓글이 기존 데이터에 반영되도록 수정
job 네이밍의 경우, verb - noun - job 의 형식으로 작성 load 와 fetch 의 경우, 로컬 소스로부터 데이터를 가져오면 load, 외부 소스로부터 데이터를 가져오면 fetch 로 구분하여 작성
컬렉션 확장함수에서 자체적으로 불변성을 보장하기 때문에 굳이 Stack 을 이용한 tmp 변수를 사용하지 않고, 모델 원본을 사용하도록 수정
…odel.refresh() 수행
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.
No description provided.