전남대 Android_최영빈_6주차 과제_step2 #82
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- 기본 코드 준비
- 구현할 기능 목록 정리
…ig's serviceState - serviceState 값이 ON_SERVICE일 때만 초기 진입 화면이 지도 화면으로 넘어감
…State is not ON_SERVICE - serviceState 값이 ON_SERVICE이 아닌 경우에는 serviceMessage 값을 초기 진입 화면 하단에 표시하고 지도 화면으로 진입하지 않음
- step2 기능 구현 목록 추가
- 알림 권한 설정
- 앱이 백그라운드 상태일 경우 FCM 기본 값을 사용하여 Notification 발생
- 앱이 포그라운드 상태일 경우 커스텀 Notification을 발생
- Notification 창을 터치하면 초기 진입 화면이 호출됨
- 5주차에서 미처 삭제하지 못한 불필요해진 테스트 파일 삭제 (참고: yb0x00/android-map-refactoring@b9171d4)
omjoonkim
approved these changes
Aug 2, 2024
| } | ||
| } | ||
|
|
||
| private fun askNotificationPermission() { |
There was a problem hiding this comment.
퍼미션에 관련된 기능은 스플래시 뿐만 아니라 여러 곳에서 사용 될 수 있습니다.
어떤 Activity 에서든 간편하게 호출이 가능하도록 로직을 컴포넌트화 시켜보세요. ㅎㅎ
|
2단계 과제도 전반적으로 코드를 잘 작성해주셨습니다.
어떤게 어려우신지 얘기해주시면 더 도움 드릴 수 있을 것 같아요.
HomeMapActivity에서 BottomSheet가 두번 inlfate되는 것 같아요. inflate대신 bind로 변경해보셔요. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
코드 작성하면서 어려웠던 점
멘토님이 중점적으로 리뷰해 주셨으면 하는 부분
+) 이전에 작성했던 "HomeMapActivityTest" 테스트 코드 실행 시 오류가 발생합니다. 어떻게 수정하면 좋을지 조언 부탁드립니다.