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

전남대 Android_이민서_6주차 과제_Step 1 #53

Open
wants to merge 4 commits into
base: leeminseo00
Choose a base branch
from

Conversation

LEEMINSEO00
Copy link

6주차 과제_Step 1

알림: 1단계 Splash Screen

코드 작성하며 어려웠던 점

  • 강의 자료와 공식 문서를 참고하여 구현하면 됐어서 크게 어려운 부분은 없었던 것 같습니다.

멘토님이 중점적으로 리뷰 해주셨으면 하는 부분

  • MVVM 아키텍처 패턴, DataBinding 등을 잘 지켰는지 점검 부탁드립니다.

val fetchSuccess: LiveData<Boolean> get() = _fetchSuccess

fun fetchRemoteConfig(remoteConfig: FirebaseRemoteConfig) {
remoteConfig.fetchAndActivate()

Choose a reason for hiding this comment

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

ViewModel에서 안드로이드에 의존적인 코드를 최대한 덜어내보세요.
이 로직의 역할을 담당하는 컴포넌트를 만들고 추상화하여 사용해보셔요.

@omjoonkim
Copy link

크게 코멘트 할 것은 없었습니다. ㅎㅎ 수고하셨어요!

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