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

[HOTFIX] 질문, 댓글 원격 알림의 알림 사운드 설정 #230

Merged
merged 1 commit into from
Oct 22, 2024

Conversation

jaewonLeeKOR
Copy link
Member

PR 타입(하나 이상의 PR 타입을 선택해주세요)

  • 기능 추가
  • 기능 삭제
  • 버그 수정
  • 의존성, 환경 변수, 빌드 관련 코드 업데이트

반영 브랜치

hotfix/#229/remoteNotiSoundless -> master

변경 사항

질문과 댓글 원격 알림이 알림 사운드가 발생하지 않던 버그를 수정했습니다.

  • 기존의 컨트롤러를 통한 원격 알림 테스트시 사용하던 @Schema(defaultValue = "default") 를 통해 알림 사운드의 기본값을 지정하던 방식으로 인해 실제 서비스 구현에서 사용하는 Builder 패턴 방식의 기본값 설정법이 맞지 않아 사운드 값이 null로 전달되었습니다.
  • @Builder.Default 를 이용해 기본 알림사운드를 설정하는 방식으로 완료했습니다.

@jaewonLeeKOR jaewonLeeKOR added the 🚨hotfix hotfix label Oct 16, 2024
@jaewonLeeKOR jaewonLeeKOR self-assigned this Oct 16, 2024
@jaewonLeeKOR jaewonLeeKOR changed the title hotfix: #229 질문, 댓글 원격 알림의 알림 사운드 설정 [HOTFIX] 질문, 댓글 원격 알림의 알림 사운드 설정 Oct 16, 2024
Copy link
Contributor

@youngeun-dev youngeun-dev left a comment

Choose a reason for hiding this comment

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

수고하셨습니다 ~ ~👍

@jaewonLeeKOR jaewonLeeKOR merged commit 87342f8 into master Oct 22, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants