-
Notifications
You must be signed in to change notification settings - Fork 30
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주차 과제 Step2 #77
base: njiyeon
Are you sure you want to change the base?
경북대 Android 남지연 6주차 과제 Step2 #77
Conversation
리베이스 해주시고 Repository Pattern Codelab 링크 드리니 참고해서 변경해보세요! https://developer.android.com/codelabs/basic-android-kotlin-training-repository-pattern?hl=ko#0 |
넵 참고해서 수정하도록 하겠습니다 !! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
지연님 6주동안 수고 많으셨습니다!
사실 Hilt, Coroutine, Room 이 세 기술 스택이 하나하나가 다 여러번 반복 숙달해야 체득이 되고 처음 배울 때 확실하게 배워야 본인만의 틀을 만드시기 쉬울거에요. 이번에 배우셨으니 제가 이전에 첨부드린 코드랩과 여러 잘 만들어진 레포들 참고해서 더 다듬어 보시면 좋을 것 같습니다!
val title = remoteMessage.notification?.title ?: "Default Title" | ||
val body = remoteMessage.notification?.body ?: "Default Body" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
소소하게 여기 Literal도 상수로 빼면 좋겠네요
넵 !! 추가적으로 기술 스택과 관련한 공부를 하면서 코드를 더 다듬어보도록 하겠습니다 ! |
과제 수행하면서 어려웠던 점 & 느낀점
firebase
사이트에 올라온 방식을 그대로 따라하니 크게 어려움은 없었습니다 !6주차 동안 과제를 수행하면서, 안드로이드 개발 전반에 있어서 많은 것을 배울 수 있었습니다.
6주차 동안 신경써서 리뷰해주신 덕분에 많은 부분을 배울 수 있었습니다! 감사합니다 !!
아래는 푸시 알람 화면입니다!