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주차 과제_Step2 #76

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

Conversation

LEEMINSEO00
Copy link

@LEEMINSEO00 LEEMINSEO00 commented Aug 2, 2024

6주차 과제_Step 2

알림: 2단계 푸시 알림

코드 작성하며 어려웠던 점

  • 강의 자료와 공식 문서를 참고하여 구현했기에 크게 어려운 부분은 없었습니다.

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

  • 기능 요구 사항&프로그래밍 요구 사항을 잘 지키면서 코드를 작성했는지 확인 부탁드립니다.

실행 결과

image image

Copy link

@omjoonkim omjoonkim left a comment

Choose a reason for hiding this comment

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

2단계 과제도 잘 진행해주셨습니다~! 수고하셨어요 ㅎㅎ

@@ -43,4 +63,50 @@ class SplashActivity : AppCompatActivity() {
}
})
}
}

private fun askNotificationPermission() {

Choose a reason for hiding this comment

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

지금은 한곳에서만 권한요구를 하지만, 서비스가 커질수록 로직이 재활용될 가능성이 높아집니다. 여러 화면에서 코드를 재활용 할 수 있도록 리팩토링을 진행햅보셔요.

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