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

[feat]: 푸시알람 구현 #39

Merged
merged 7 commits into from
Feb 1, 2024
Merged

[feat]: 푸시알람 구현 #39

merged 7 commits into from
Feb 1, 2024

Conversation

realisshomyang
Copy link
Contributor

PULL REQUEST

🎋 작업중인 브랜치

💡 작업동기

  • 퇴근 시 알람을 위한 푸시알람 기능이 필요해서 작업하게됐습니다!

🔑 주요 변경사항

  • 푸시알람 관련 application.yml을 노션에 업데이트하였습니다!
  • resources에 json하나 업로드 해야하는데, 노션에 업로드해놓겠습니다! 다운받아서 사용하시길 바랍니다.

💡 관련 이슈

@realisshomyang realisshomyang added the ✨ feat 새로운 기능 추가 label Jan 31, 2024
@realisshomyang realisshomyang self-assigned this Jan 31, 2024
@realisshomyang realisshomyang linked an issue Jan 31, 2024 that may be closed by this pull request
3 tasks
Copy link
Contributor

@wu-seong wu-seong left a comment

Choose a reason for hiding this comment

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

쉽지 않았을 것 같은데 고생하셨습니다~~

for (User user : userList) {
NotificationMessage notificationMessage = NotificationMessage.toGoHomeNotification(user);
Message message = fcmService.createMessage(user.getFcmToken(), notificationMessage);
fcmService.send(message);
Copy link
Contributor

Choose a reason for hiding this comment

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

궁금해서 그런데 여기서 가져온 fcmToken은 푸시 알림 동의 했을때 set 되는건가요??

Copy link
Contributor Author

Choose a reason for hiding this comment

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

네 온보딩에서 푸시 알람 동의했을 때 프론트 측에서 한 번에 유저 정보 업데이트 api 요청할 때 담아서 보내주고, 이를 기반으로 백이 디비에 업데이트 하는 식으로 로직 이해하고 있습니다 ~

Copy link
Contributor

@kimjm9841 kimjm9841 left a comment

Choose a reason for hiding this comment

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

수고 많으셨습니다! 👍

) {
public static NotificationMessage toGoHomeNotification(User user) {
return NotificationMessage.builder()
.title(user.getName() + "님, 이제 퇴근하실 시간이에요.")
Copy link
Contributor

Choose a reason for hiding this comment

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

별 건 아니지만 WF 보면 다 닉네임으로 부르길래 여기도 name 대신 nickname 쓰는 게 더 좋지 않을까 싶어용

Copy link
Contributor Author

Choose a reason for hiding this comment

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

넵 수정 후 커밋할게요!

@realisshomyang
Copy link
Contributor Author

Merge할게요 !

@realisshomyang realisshomyang merged commit e11af62 into dev Feb 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ feat 새로운 기능 추가
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[feat] : 푸시알림 구현
3 participants