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

[Feat/YAP-63] 온보딩 시스템 알림권한 설정을 구현합니다. #50

Merged
merged 4 commits into from
Jan 20, 2025

Conversation

MoonsuKang
Copy link
Member

Related issue 🛠

closed #<issue_number>

어떤 변경사항이 있었나요?

  • 🐞 BugFix Something isn't working
  • 🎨 Design Markup & styling
  • 📃 Docs Documentation writing and editing (README.md, etc.)
  • ✨ Feature Feature
  • 🔨 Refactor Code refactoring
  • ⚙️ Setting Development environment setup
  • ✅ Test Test related (Junit, etc.)

CheckPoint ✅

PR이 다음 요구 사항을 충족하는지 확인하세요.

  • PR 컨벤션에 맞게 작성했습니다. (필수)
  • merge할 브랜치의 위치를 확인해 주세요(main❌/develop⭕) (필수)
  • Approve된 PR은 assigner가 머지하고, 수정 요청이 온 경우 수정 후 다시 push를 합니다. (필수)
  • BugFix의 경우, 버그의 원인을 파악하였습니다. (선택)

Work Description ✏️

  • accompanist library를 활용하여 시스템 알림권한 요청 구현
  • 알림 추후 설정 및 OS 설정으로 이동 구현
systemnoti.mp4

Uncompleted Tasks 😅

  • Task1

To Reviewers 📢

지금 까지 했던 모든 기능 중 제일 오래 걸렸습니다. 능지 이슈 때문에..
무슨 방법을 써도 계속 팝업이 안떠서 build-logic module을 다시 되돌아 보기 시작했습니다.
target sdk가 없더라구요. 하지만 저는 이게 문제가 될 지 몰랐습니다 허허...
모든 구글링을 통해서 targetsdk 문제로 수렴하는거 같아 다음과 같은 command를 terminal에 입력해봤습니다.
./gradlew :app:processDebugManifest
이 커맨드를 입력하면 app/build/intermediates/merged_manifests/debug/AndroidManifest.xml 이 경로에 모든 정보가 담기게 되는데요.
image

targetsdk가 28로 잡혀서 안뜨는 것이었습니다..
screen단에서 Sdk로그를 찍으면 기기의 sdk버전인 33만 떠서 확인할 수 없습니다.
정말 코드 한 줄 때문에 3시간은 넘게 쓴거 같네요. ㅋㅋㅋㅋㅋㅋㅋ;; 생각지도 못했습니다.

@DongChyeon
Copy link
Member

역시 디버깅이 답이군요 고생하셨습니다.

Copy link
Member

@DongChyeon DongChyeon left a comment

Choose a reason for hiding this comment

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

굿굿!

@MoonsuKang MoonsuKang merged commit aa1dd97 into develop Jan 20, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants