compose migration : 로그인, 회원가입, 홈, 관심 목록 마이그레이션#117
Merged
Conversation
- LoginFragment를 BaseComposeFragment로 전환 (기존 BaseFragment를 상속하는 다른 Fragment의 점진적 마이그레이션을 위해 viewBinding 외의 처리들을 CoreFragment로 이전) - ViewModel에 UiState 및 Flow 도입 - 비즈니스 로직을 ViewModel로 이전
- 앱 바 내부 Text 컴포저블의 타입을 구분하는 enum class 선언
-ViewModel 로직 응집도 향상 (상태 업데이트 및 유효성 검사) - Fragment의 역할을 UI 렌더링 및 SideEffect 처리로 제한 - Screen Composable에서의 불필요한 콜백 파라미터 제거 및 uiState/onEvent로 통합
- 카카오 로그인 SDK 함수 실행 시 내부적으로 새 Activity에 띄워야하는데 Application Context가 전달되어 Activity 띄우기 실패 -> Application Context 대신 Activity의 context를 전달
- 구단 Id 리스트를 해당하는 구단 명으로 바꿔 하나의 문자열로 출력되도록 변경
- 3개의 home filter bottom sheet를 타입별로 세분화해 1개의 ModalBottomSheet composable 재사용하여 구현 - 아이템 삭제 시 setFragmentResultListener로 boardId를 전달받는 대신 Compose Navigation Result를 사용해 boardId를 전달받아 아이템 삭제
- 좋아요 api 응답 수정 - 관심 목록 LazyColumn 항목 삭제 애니메이션 추가 - 관심 목록 fragment 기능 구현
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
작업 내용