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

출시 후 해야할 일 #70

Open
17 of 25 tasks
easyhooon opened this issue Aug 17, 2023 · 10 comments
Open
17 of 25 tasks

출시 후 해야할 일 #70

easyhooon opened this issue Aug 17, 2023 · 10 comments
Assignees
Labels
documentation Improvements or additions to documentation

Comments

@easyhooon
Copy link
Collaborator

easyhooon commented Aug 17, 2023

  • 플레이스토어 그래픽 이미지 교체
  • README 꾸미기(플레이스토어 링크,그래픽 이미지, 앱 이미지, 앱 소개, 사용한 아키텍처에 대한 설명, 사용한 기술 스택, etc)
  • debug, release 앱 분리
  • 코드 리뷰 요청(리뷰는 많은 분들에게 받을수록 좋다. 커피챗, 유로 커피챗 앱, 지인, 스터디 등을 통해)
  • 코드 리뷰 피드백
  • 리팩토링(너무 많은 역할을 수행하는 함수 분리, Recomposition 최적화 적용, UiState 핸들링 개선) ~ing
    • UiState 를 data class 에서 sealed class 또는 sealed interface 를 통해 관리, onEvent 함수를 통해 event 를 관리
  • 사용성 테스트(테스크를 입력할때, 현재 목표 텍스트필드가 비어있을 경우 바텀시트가 올라올때 포커스가 목표입력으로 할당되면서 키보드가 자동으로 올라오는데 이에 대한 UX 의견 묻기)
  • Api Request 가 TimeOut 이 되는 이슈 해결(한번 TimeOut이 발생하면, 앱이 먹통이 되는 이슈(다시 시도도 안됨)
  • 필요없는 커스텀 컴포저블 함수들 제거(TitleText 같은 컴포저블 함수가 굳이 필요한지 논의)
  • core/ui 모듈에 넣을 파일과 feature/home 모듈에 넣을 파일 기준을 정하여 재배치
  • 파일 네이밍 수정 ~ ing
  • ViewModel uiState 변경 로직 통일(.value vs .update)
  • 데이트 피커 선택 박스가 기기에 상관없이 날짜 텍스트에 정중앙에 위치하도록 수정
  • github action + fastlane 을 통한 cd 설정 ~ing -> Bitrise 로 migration 예정
  • 스플래시 스크린 커스텀
  • Module Graph 생성
  • in-app update 적용(optional)
  • baseline profile 설정해보기(macrobenchmark 설정) ~ing
  • 추가 기능 논의(optional, chatGPT 를 이용한 메인 목표와 관련된 테스크 추천) ~ing
  • 테스트 코드 작성 ~ing
  • hilt kapt -> ksp 로 migration
  • compose-metrics 적용
  • bottomsheet Composable 분리, 및 상태 관리 개선 ~ ing
  • 태블릿 대응
  • component, screen 프리뷰 구성
@easyhooon
Copy link
Collaborator Author

easyhooon commented Aug 17, 2023

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

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

@easyhooon easyhooon self-assigned this Aug 17, 2023
@easyhooon easyhooon added the documentation Improvements or additions to documentation label Aug 17, 2023
@easyhooon
Copy link
Collaborator Author

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
Copy link
Collaborator Author

ksp migration

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

@easyhooon
Copy link
Collaborator Author

easyhooon commented Sep 18, 2023

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

@easyhooon
Copy link
Collaborator Author

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

@easyhooon
Copy link
Collaborator Author

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

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

No branches or pull requests

1 participant