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

MNG-02 알림센터에서 푸시알림을 클릭했을 때 이동하게 수정 #212

Merged
merged 3 commits into from
Jan 30, 2024

Conversation

minsu20
Copy link
Member

@minsu20 minsu20 commented Jan 30, 2024

PR 타입

  • 기능 추가
  • 버그 수정
  • 의존성, 환경 변수, 빌드 관련 업데이트
  • 기타 사소한 수정

개요

  • 알림 센터에서 푸시알림을 클릭했을 떄 이동하게 수정

변경 사항

  • 알림 센터에서 푸시알림을 클릭했을 떄 이동할 수 있게 addData추가
  • fcm 부분에서 추상화게 의존할 수 있게 함수 구조 리팩토링
    -> 우선 기존 함수는 주석 처리했고, 반영 후 리마인드 알림까지 잘 보내지면 후에 삭제할 예정 !

코드 리뷰 시 참고 사항

  • fcm 부분에서 개방-폐쇄 원칙을 지킬 수 있게 추상화게 의존할 수 있게 했는데 이에 대해 봐주시면 감사하겠습니당

@minsu20 minsu20 merged commit 1bfb162 into main Jan 30, 2024
1 check passed
@minsu20 minsu20 mentioned this pull request Jan 30, 2024
4 tasks
@minsu20 minsu20 deleted the MNG-02 branch January 30, 2024 02:33
@seungueonn
Copy link
Member

오 추상화 의존 .. 확인했습니다! 문제 없이 알림 잘 가지 않을까요 ??? 고생하셨어요 !!!

@minsu20
Copy link
Member Author

minsu20 commented Jan 31, 2024

�요새 오브젝트라는 책을 읽고 있는데 이 책이 리팩토링할 때의 가이드를 제공해주는 느낌이에요 ㅎㅎ

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