Nexters / BandalArt-Android

부담 없는 만다라트 계획표로 당신의 목표를 더욱 선명하게, 반다라트 Android
https://play.google.com/store/apps/details?id=com.nexters.bandalart
23 stars 0 forks source link

출시 후 해야할 일 #70

Open easyhooon opened 1 year ago

easyhooon commented 1 year ago
easyhooon commented 1 year ago
  • core/ui 모듈에 넣을 파일과 feature/home 모듈에 넣을 파일 기준을 정하여 재배치
  • 파일 네이밍 수정

이거 넥나잇에 할 일에 있던거고 형이 좀 분리해놨긴 했는데, 같이 얘기는 안했던거 같아서 넣어놨어

easyhooon commented 1 year ago

리팩토링(너무 많은 역할을 수행하는 함수 분리, Recomposition 최적화 적용, UiState 핸들링 개선)

https://github.com/droidknights/DroidKnights2023_App/pull/227

https://sungbin.land/a-deep-dive-into-jetpack-compose-stability-38b5b109da71

https://getstream.io/blog/jetpack-compose-guidelines/

https://medium.com/androiddevelopers/jetpack-compose-stability-explained-79c10db270c8

https://issuetracker.google.com/issues/238057117

활용 예시 https://github.com/mash-up-kr/SulSul-Android/blob/develop/presentation/src/main/java/com/mashup/alcoholfree/presentation/utils/ImmutableWrapper.kt -> 굳이 Wrapper 클래스를 생성하지 않아도 List.toImmutableList() 함수로 ImmutableList 로 변환 해줄 수 있었음

https://github.com/android/compose-samples/blob/main/Jetcaster/app/src/main/java/com/example/jetcaster/ui/home/HomeViewModel.kt

easyhooon commented 1 year ago

Module Graph 생성

https://github.com/droidknights/DroidKnights2023_App 드로이드나이츠의 Module Graph 만드는 방법을 적용해보려고 했으나, Deprecated Gradle features were used in this build, making it incompatible with Gradle 9.0. 해당 에러로 인해 Build Failed

easyhooon commented 1 year ago

cd setup

reference) https://jsieun73.tistory.com/183 https://jsieun73.tistory.com/190?category=611190 https://docs.fastlane.tools/getting-started/android/setup/ cd를 위한 기본 설정 https://blog.mint-soft.com/5 fastlane에 파이어베이스 distribution 설정 시 (음,, 그리 중요하진 않을 수도) android-cd.yml 작성 fastlane 설치 시 이슈 해결 방법

이거 두개 말고도 참고한 글이 있으면 추가 바람

easyhooon commented 1 year ago

compose metrics setup

references) https://chrisbanes.me/posts/composable-metrics/ https://velog.io/@ams770/jetpack-compose-skippable-issue-2-metrics https://sungbin.land/jetpack-compose-%EC%BB%B4%ED%8C%8C%EC%9D%BC%EB%9F%AC-%EB%94%94%EB%B2%84%EA%B9%85-cf21ce431387

easyhooon commented 1 year ago

ksp migration

references) https://www.charlezz.com/?p=45255

easyhooon commented 1 year ago

현재는 반다라트 목록을 persistentList로 선언해둔 상태인데 그 리스트의 변화가 빈번하게 발생하지 않으므로 immutableList 를 사용하는게 더 적절할듯

easyhooon commented 1 year ago

하위 컴포저블에 uiState 를 전달해주지 않고 uiState.title 과 같이 필요한 변수를 전달하는 방향으로 해서 하위 컴포저블이 안정하도록 변경

easyhooon commented 1 year ago

bottom sheet state를 Top Level 에서 관리하는 방법

https://github.com/mash-up-kr/ssam-d-Android/blob/develop/presentation/src/main/java/com/mashup/presentation/KeyLinkApp.kt

https://github.com/mash-up-kr/ssam-d-Android/blob/develop/presentation/src/main/java/com/mashup/presentation/KeyLinkAppState.kt

easyhooon commented 1 year ago

https://www.youtube.com/watch?v=njchj9d_Lf8&t=774s

이벤트(One-Time-Event) 와 상태(State)를 구분할 것(분리)