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

[PC-691] FCM 연결 #88

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from
Open

[PC-691] FCM 연결 #88

wants to merge 2 commits into from

Conversation

tgyuuAn
Copy link
Member

@tgyuuAn tgyuuAn commented Mar 3, 2025

1. ⭐️ 변경된 내용

  • FCM 연결

2. 📌 이 부분은 꼭 봐주세요!

이전 프로젝트에서 문서화 한 것이에용 FCM 참고해주세요!

image




Service In CleanArchitecture

서비스는 안드로이드 프레임워크 의존성이 있는 레이어이므로, Domain을 제외한 Data, Presentation Layer에 위치할 수 있는데,

Service도 Activity와 마찬가지로 4대 컴포넌트이고 앱의 시작점이므로 Presentation Layer가 더 적합하다고 판단하였습니다.

App 모듈에도 둘 수 있으나 App 모듈과 presentation 모듈을 분리했음으로,

App모듈은 라이브러리의 시작 및 DI(IoC 컨테이너)의 역할로 사용하는 것이 더 타당하다고 생각하여 Presentation Layer에 위치시켰습니다!!

@tgyuuAn tgyuuAn requested a review from sksowk156 March 3, 2025 15:51
@tgyuuAn tgyuuAn self-assigned this Mar 3, 2025
@tgyuuAn tgyuuAn added 기능 ⚒️ 새로운 기능 구현 ⚒️ 리뷰 원해요🔥 피어의 리뷰를 기다리는 ing.. 🔥 ㅌㄱ태규 ☀️ 훗날 크게될 ENFP 남성, tgyuuAn labels Mar 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ㅌㄱ태규 ☀️ 훗날 크게될 ENFP 남성, tgyuuAn 기능 ⚒️ 새로운 기능 구현 ⚒️ 리뷰 원해요🔥 피어의 리뷰를 기다리는 ing.. 🔥
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant