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

[Feature/391] 기기 등록 / 푸시 알림 설정 조회 & 수정 API 구현 #396

Merged
merged 17 commits into from
Oct 28, 2024

Conversation

joowojr
Copy link
Contributor

@joowojr joowojr commented Oct 27, 2024

Type of change

  • Feature : 새로운 기능 추가
  • Bug fix : 버그 수정
  • Refactor : 코드 리팩토링 작업
  • Document : 문서작업
  • Test : 테스트 코드 작성 및 테스트 작업
  • Style : 코드 스타일 및 포맷팅 작업
  • CI/CD : CI/CD 작업 수정
  • Chore : 패키지 매니저, 라이브러리 업데이트 등의 작업

PR Desciption

변경 사항 설명

  • 기기 정보 등록 및 푸시 알림 활성화 API 구현

    • 푸시 알림 최초 설정 시에 사용
  • 푸시 알림 설정 조회 API 구현

  • 푸시 알림 설정 수정 API구현

  • 기획 구체화에 따른 푸시 알림 유형 추가 및 메세지 템플릿 수정

    • NotificationType.java
    • NotificationMessageGenerator.java

Requirements for Reviewer

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

  • 기본이 되는 API를 구현하고 기존의 API를 리팩토링 하였습니다.
  • 이번 PR 머지 후, notification 도메인에 관련된 비즈니스 로직을 추가 구현 및 수정할 예정입니다.

PR Log

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

새롭게 배운 것

고민 중인 사항

첨부 자료

관련 이슈

@github-actions github-actions bot added the ✨ feature 기능 구현 관련 label Oct 27, 2024
@joowojr joowojr requested a review from hosunglee222 October 27, 2024 02:48
@github-actions github-actions bot added the ♻️ refactor 리펙터링 관련 label Oct 27, 2024
Copy link
Member

@hosunglee222 hosunglee222 left a comment

Choose a reason for hiding this comment

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

고생하셨습니다! 리뷰확인해주세요

@joowojr joowojr merged commit d7588d8 into dev Oct 28, 2024
1 check passed
@joowojr joowojr deleted the feature/391 branch October 28, 2024 03:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
♻️ refactor 리펙터링 관련 ✨ feature 기능 구현 관련
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feature] 푸시 알림 설정 API 구현
2 participants