Skip to content

Conversation

@angryPodo
Copy link
Member

PR chekList

  • ktLint 포맷을 지킨다.
  • indent(인덴트, 들여쓰기) depth를 3이 넘지 않도록 구현한다.
  • 함수(또는 메서드)가 한 가지 일만 하도록 최대한 작게 만든다.
  • class를 최대한 작게 만든다.
  • else를 지양한다(얼리 리턴 사용).

Related issue 🛠

Work Description ✏️

  • 로컬 푸시 알림 기능 구현

    • WorkManager와 Hilt를 사용하여 정해진 시간에 일간/주간 알림을 발송합니다.
    • 아키텍처 원칙에 따라 의존성을 격리하고 확장성을 고려하여 설계했습니다.
  • 모듈 추가 및 설정

    • :core:notification: 알림 UI(채널, 빌더) 로직을 담당하는 모듈을 추가했습니다.
    • :core:work: WorkManager 스케줄링 및 Worker 로직을 담당하는 모듈을 추가했습니다.
    • :app 모듈이 WorkManager에 직접 의존하지 않도록, :core:work에서 수동 초기화를 담당하는 구조로 구현했습니다.
  • 주요 구현 내용

    • HilingualNotificationManager: 알림 생성을 캡슐화한 싱글톤 클래스.
    • DailyNotificationWorker / WeeklyNotificationWorker: HiltWorker를 사용하여 각 알림 발송 로직 구현.
    • HilingualWorkManagerConfigurator: 앱 시작 시 WorkManager를 초기화하고 작업을 스케줄링.
    • 알림 아이콘이 규격에 맞도록 투명 배경의 단색 벡터로 수정.

Screenshot 📸

일간 알림 주간 알림
image image

Uncompleted Tasks 😅

  • 없음.

To Reviewers 📢

  • WorkManager의 초기화 로직을 :core:work 모듈로 격리하여 :app 모듈의 의존성을 깨끗하게 유지하는 구조에 대해 중점적으로 봐주시면 감사하겠습니다.

@angryPodo angryPodo self-assigned this Dec 24, 2025
@angryPodo angryPodo requested a review from a team as a code owner December 24, 2025 14:03
@angryPodo angryPodo requested review from Daljyeong, Hyobeen-Park, nahy-512 and nhyeonii and removed request for a team December 24, 2025 14:03
@angryPodo angryPodo added FEAT✨ 사용자에게 제공되는 새로운 기능 구현 및 동작 변경 🍀민재 민재 담당 labels Dec 24, 2025
@angryPodo angryPodo linked an issue Dec 24, 2025 that may be closed by this pull request
Copy link
Collaborator

@nhyeonii nhyeonii left a comment

Choose a reason for hiding this comment

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

코드 확인 완료했습니다 !! 푸쉬 알림,, 생소해서 그런지 아직 어렵네요...ㅎㅎ 혹시 리드님께서 만들어두신 딥링크 기반 라이브러리는 아직 적용이 안 되어있는 것 같은데 추후 적용 예정이 있으신지 궁금합니다 !! 작업하시느라 너무 고생 많으셨습니다..ㅜㅜ 항상 많이 배워가요 감사합니다

@angryPodo
Copy link
Member Author

코드 확인 완료했습니다 !! 푸쉬 알림,, 생소해서 그런지 아직 어렵네요...ㅎㅎ 혹시 리드님께서 만들어두신 딥링크 기반 라이브러리는 아직 적용이 안 되어있는 것 같은데 추후 적용 예정이 있으신지 궁금합니다 !! 작업하시느라 너무 고생 많으셨습니다..ㅜㅜ 항상 많이 배워가요 감사합니다

네! 아직 멀티모듈을 지원하는 기능을 넣지 않아서 적용하지 않았어요. 그리고 이번 볼륨에서는 단순히 홈화면으로 진입하기에 Intent를 앱시작으로 하면 되었습니다ㅎㅎ 추후 FCM을 통해서 의도적인 백스택을 생성할때 적용할 예정이에요

@angryPodo angryPodo merged commit 8987bc3 into develop Dec 27, 2025
2 checks passed
@angryPodo angryPodo deleted the feat/local-notification branch December 27, 2025 14:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

FEAT✨ 사용자에게 제공되는 새로운 기능 구현 및 동작 변경 🍀민재 민재 담당

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[FEAT] 로컬 푸시 알림 구현

3 participants