#76 [FEAT] fcm push 구현, 미션 성공시 fcm push 전송

[hye-on]

☀️ 작업 사항

• 모두에게 push 보내기 api (공지용)
• 미션 수행시 그룹원들에게 push 보내기

  ☀️ 관련 이슈

  related #76

☀️ 참고 사항

• sql 수정 있습니다.
• 전에 있던 코드는 주석처리 했는데 삭제해도 괜찮을지 알려주세요!

[hye-on]

리뷰 작성하였습니다!

주석 처리하신 부분은 NotificationProvider과 PushLogService를 제외하고 삭제하셔도 괜찮을 것 같습니다. 두 클래스는 추후 추가기능에 사용될 가능성이 있는 것 같습니다.

아하 혹시 서버에는 사용중인 코드만 남겨놓는것이 어떨까요! 혹시 나중에 사용하게 된다면 새로 생성한는 것을 건의해봅니다ㅏ!

[hye-on]

리뷰 작성하였습니다! 주석 처리하신 부분은 NotificationProvider과 PushLogService를 제외하고 삭제하셔도 괜찮을 것 같습니다. 두 클래스는 추후 추가기능에 사용될 가능성이 있는 것 같습니다.

아하 혹시 서버에는 사용중인 코드만 남겨놓는것이 어떨까요! 혹시 나중에 사용하게 된다면 새로 생성한는 것을 건의해봅니다ㅏ!

의견주신대로 두 클래스 파일 남겨놓았습니다!

[05AM]

리뷰 작성하였습니다! 주석 처리하신 부분은 NotificationProvider과 PushLogService를 제외하고 삭제하셔도 괜찮을 것 같습니다. 두 클래스는 추후 추가기능에 사용될 가능성이 있는 것 같습니다.

아하 혹시 서버에는 사용중인 코드만 남겨놓는것이 어떨까요! 혹시 나중에 사용하게 된다면 새로 생성한는 것을 건의해봅니다ㅏ!

좋은 의견이라고 생각하지만 이미 구현된 부분을 삭제할 필요가 있을까요? 알림 로그를 기록하는데 특별한 추가 기능이 필요하지 않고, 알림 기록이 필요한 기능이라고 생각해서 다음 버전 업 때 들어가면 좋겠다고 생각하고 있습니다. 이후에 리팩토링 하면서 수정하는 것은 어떨까요?

[hye-on]

리뷰 작성하였습니다! 주석 처리하신 부분은 NotificationProvider과 PushLogService를 제외하고 삭제하셔도 괜찮을 것 같습니다. 두 클래스는 추후 추가기능에 사용될 가능성이 있는 것 같습니다.

아하 혹시 서버에는 사용중인 코드만 남겨놓는것이 어떨까요! 혹시 나중에 사용하게 된다면 새로 생성한는 것을 건의해봅니다ㅏ!

좋은 의견이라고 생각하지만 이미 구현된 부분을 삭제할 필요가 있을까요? 알림 로그를 기록하는데 특별한 추가 기능이 필요하지 않고, 알림 기록이 필요한 기능이라고 생각해서 다음 버전 업 때 들어가면 좋겠다고 생각하고 있습니다. 이후에 리팩토링 하면서 수정하는 것은 어떨까요?

리뷰 작성하였습니다! 주석 처리하신 부분은 NotificationProvider과 PushLogService를 제외하고 삭제하셔도 괜찮을 것 같습니다. 두 클래스는 추후 추가기능에 사용될 가능성이 있는 것 같습니다.

아하 혹시 서버에는 사용중인 코드만 남겨놓는것이 어떨까요! 혹시 나중에 사용하게 된다면 새로 생성한는 것을 건의해봅니다ㅏ!

좋은 의견이라고 생각하지만 이미 구현된 부분을 삭제할 필요가 있을까요? 알림 로그를 기록하는데 특별한 추가 기능이 필요하지 않고, 알림 기록이 필요한 기능이라고 생각해서 다음 버전 업 때 들어가면 좋겠다고 생각하고 있습니다. 이후에 리팩토링 하면서 수정하는 것은 어떨까요?

네 좋습니다~~!