Skip to content

Conversation

@starboxxxx
Copy link
Collaborator

@starboxxxx starboxxxx commented Feb 4, 2025

📄 요약(Summary)

아지트 푸시 알림 logic 변경

✍🏼 상세(More)

PR Desciption

변경 사항 설명

아지트 푸시 알림
게시글로 푸시알림 : 작업 생성
=> 작업 생성시 작업 생성 푸시 알림을 webhook url을 통해 아지트로 보내면 아지트에서 해당 푸시 알림을 생성한 후 반환 값으로 해당 푸시 알림 게시글의 ID를 줍니다.
=> 해당 게시글 ID를 Task의 agitPostId에 저장
작업 요청 알림 게시글의 댓글로 알림 : 작업 승인, 작업 처리자 변경, 작업 상태 변경
=> 해당 작업의 agitPostId를 조회 후 이를 payload에 포함하여 webhook url을 통해 아지트로 보내면 아지트에서 해당 알림 게시글의 댓글로 푸시 알림을 생성

카카오워크 & 이메일 푸시 알림
작업 생성, 작업 승인, 작업 처리자 변경, 작업 상태 변경, 댓글 작성에 대해 푸시 알림 기능 제공

Requirements for Reviewer

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요
ex) 메서드 XXX의 이름을 더 잘 짓고 싶은데 혹시 좋은 명칭이 있을까요?

PR Log

PR 작업하면서 고민했던 내용, 해결한 내용, 고민 중인 내용 등

새롭게 배운 것

고민 중인 사항

첨부 자료

스크린샷 2025-02-04 오후 2 52 24
  • 작업 생성 알림의 경우 게시글 알림으로 전송
  • 그 외의 알림의 경우(댓글 알림은 아지트 알림으로 전송 X) 해당 게시글의 댓글 알림으로 전송

Requirements for Reviewer

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요

✅ 체크리스트(Checklist)

  • [ ✅ ] PR 양식에 맞게 작성했습니다
  • [ ✅ ] 모든 테스트가 통과했습니다
  • [ ✅ ] 프로그램이 정상적으로 작동합니다
  • [ ✅ ] 적절한 PR 라벨을 설정했습니다
  • [ ✅ ] 불필요한 코드를 제거했습니다

🚪 이슈 번호(Issue numbers)

Closes #{이슈 번호}

@starboxxxx starboxxxx added ✨ feature 구현·개선 사항에 관련된 내용입니다 MID 우선순위 중 labels Feb 4, 2025
@starboxxxx starboxxxx self-assigned this Feb 4, 2025
@starboxxxx starboxxxx requested a review from joowojr February 4, 2025 06:46
Copy link
Contributor

@joowojr joowojr left a comment

Choose a reason for hiding this comment

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

수고하셨습니다! 리뷰 확인 부탁드리고 추후 알림 템플릿에 대해 기획을 더 해보아야 할 것같습니다.

@starboxxxx starboxxxx requested a review from joowojr February 4, 2025 14:34
@starboxxxx
Copy link
Collaborator Author

starboxxxx commented Feb 4, 2025

수고하셨습니다! 리뷰 확인 부탁드리고 추후 알림 템플릿에 대해 기획을 더 해보아야 할 것같습니다.

@joowojr 수정 완료하였습니다! 확인 부탁드립니다!

@starboxxxx starboxxxx force-pushed the CLAP-257 branch 3 times, most recently from 3de68ff to 8fb5fff Compare February 4, 2025 16:21
@starboxxxx starboxxxx merged commit c1e2b4e into develop Feb 4, 2025
1 check passed
@starboxxxx starboxxxx deleted the CLAP-257 branch February 4, 2025 17:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

✨ feature 구현·개선 사항에 관련된 내용입니다 MID 우선순위 중

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants