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

[FIX]fcm관련 로직 수정 #232

Merged
merged 1 commit into from
May 24, 2024
Merged

[FIX]fcm관련 로직 수정 #232

merged 1 commit into from
May 24, 2024

Conversation

Hong0329
Copy link
Contributor

@Hong0329 Hong0329 commented May 24, 2024

📣 Related Issue

📝 Summary

  • 본인이 본인 관련된 글들에 대해 좋아요, 답글 남길 시에 푸시 알림 발생 안하도록 로직 수정
  • fcm 데이터 페이로드에는 모든 값이 문자열 형식이어야 한다는 말이 있어서 일단 수정해보겠습니다. 실제로 Long일 경우에 관련 에러가 발생했습니다.

🙏 Question & PR point

  • 클라 테스트를 위해서 일단 수정 사항 bypass머지하겠습니다. 양해부탁드립니다 ㅜ

📬 Postman

image

@Hong0329 Hong0329 requested a review from bo-ram-bo-ram May 24, 2024 08:23
@Hong0329 Hong0329 self-assigned this May 24, 2024
@Hong0329 Hong0329 merged commit a6f00c1 into develop May 24, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FIX] fcm관련 로직 수정
1 participant