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

kuring-70 common:util, common:ui_util 모듈 분리 #73

Merged
merged 5 commits into from
Sep 18, 2023

Conversation

mwy3055
Copy link
Member

@mwy3055 mwy3055 commented Sep 16, 2023

Jira 티켓 링크

커밋 내용

common 모듈 중 util, ui_util 모듈을 분리했습니다. 분리한 파일은 다음과 같습니다.

common:util

  • util (FcmUtil, PreferenceUtil, modified_external_libraries 제외)
  • di.DispatcherModule
  • di.NetworkModule 일부 (Retrofit 객체를 @Provides하는 코드)
  • data.api.HeaderInterceptor (요청 헤더에 Kuring, Android 버전을 넣는 코드)

common:ui_util

  • adapter.BindingAdapters
  • ui.customview
  • ui.dialogs
  • ui.SingleLiveEvent
  • util.modified_external_libraries
  • util.ContextExt
  • 개별 feature 모듈에 속할 레이아웃을 제외한 모든 리소스 파일

아래는 data.domain 모듈화 완료 후 옮길 파일입니다.

  • ui.compose
  • KuringNavigator

참고: Android 모듈화 계획 (Notion)

Copy link
Member

@yeon-kyu yeon-kyu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM~!

@mwy3055 mwy3055 merged commit c767982 into task/kuring-80 Sep 18, 2023
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants