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

[ST-660] 백엔드 FCM 메시지 포맷 반영 #73

Merged
merged 2 commits into from
Oct 4, 2023
Merged

[ST-660] 백엔드 FCM 메시지 포맷 반영 #73

merged 2 commits into from
Oct 4, 2023

Conversation

w8385
Copy link
Member

@w8385 w8385 commented Oct 3, 2023

업데이트 유형

  • Hot Fix
  • Release
  • Develop
  • Others

업데이트 개요

업데이트 요약

  • FCM 메시지 포맷 반영
    푸시 메시지 전송시 아래 포맷 반영
    소켓 메시지와 동일하나, type 추가 (소켓에도 추가해야할지도?)
"data": {
  "chattingId": "채팅방 id",
  "sender": "송산자 id",
  "format": "메시지 포맷",
  "body": "메시지 본문",
  "createdAt": "보낸 시각",
  "type": "메시지 타입 (chatting, announce etc)",
},
  • socket - gateway / repository 분리
    단순 메시지 전송 메소드 socket.repository로 이동
    question, offer, tutoring에서 socket.gateway가 아닌 socket.repository를 참조하도록 수정

해결한 문제

소켓&FCM 메시지 포맷 얼추 통일
다른 모듈에서 socket.gateway를 참조하는 부분 수정

미해결 문제

테스트

  • 유닛 테스트
  • 빌드 테스트 (통합 테스트)
  • 기타 유효성 테스트

수정 사항 진단

  • 코드가 이 프로젝트의 스타일 지침을 따릅니다.
  • 코드에 대한 자체 리뷰를 수행했습니다.
  • 변경 내역에 대해 주석을 작성했습니다.
  • 해당 PR에 대한 문서를 변경했습니다.
  • 기능, 수정 사항, 이슈에 대한 테스트 코드를 추가했습니다.
  • 변경 내용이 로컬 개발환경에서의 테스트를 통과했습니다.

@w8385 w8385 requested a review from seongyunlee October 3, 2023 08:13
@notion-workspace
Copy link

Copy link
Contributor

@seongyunlee seongyunlee left a comment

Choose a reason for hiding this comment

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

Clean~!

@w8385 w8385 merged commit 81b41bb into dev Oct 4, 2023
@w8385 w8385 deleted the fix/fcm branch October 4, 2023 07:57
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