Skip to content

#395 [Feat] 관리자용 푸시 알림 발송 테스트 API 추가 - #398

Merged
si-zero merged 2 commits into
devfrom
feat/#395
Aug 16, 2026
Merged

#395 [Feat] 관리자용 푸시 알림 발송 테스트 API 추가#398
si-zero merged 2 commits into
devfrom
feat/#395

Conversation

@si-zero

@si-zero si-zero commented Aug 16, 2026

Copy link
Copy Markdown
Member

#️⃣ 연관된 이슈

📝 작업 내용

✨ Feat

내용 파일
관리자가 특정 유저 디바이스에 알림 설정(ON/OFF) 무관하게 즉시 테스트 푸시를 보낼 수 있는 API 추가 AdminNotificationController.java, AdminNotificationTestRequest.java, NotificationDispatchService.java

📌 공유 사항

  1. POST /api/v1/admin/notices/test (userId, title, body) — ROLE_ADMIN 권한 필요.
  2. 테스트 중 FcmPushService가 title/body를 무시하는 별도 버그를 발견해 🐛 Fix: Android FCM 푸시에 title/body가 실리지 않아 알림이 안 뜨는 버그 수정 #396(PR #396 [Fix] Android FCM 푸시에 title/body가 실리지 않아 알림이 안 뜨던 문제 수정 #397)으로 분리 처리함(Android 알림 미노출 원인).

✅ 체크리스트

  • Reviewer에 팀원들을 선택했나요?
  • Assignees에 본인을 선택했나요?
  • 컨벤션에 맞는 Type을 선택했나요?
  • Development에 이슈를 연동했나요?
  • Merge 하려는 브랜치가 올바르게 설정되어 있나요?
  • 컨벤션을 지키고 있나요?
  • 로컬에서 실행했을 때 에러가 발생하지 않나요?
  • 팀원들에게 PR 링크 공유를 했나요?

📸 스크린샷

AdminNotificationTestPushIntegrationTest로 admin 토큰 발송 → FcmPushService.send() 호출 및 인자(title/body/device) 검증 완료 (테스트 통과).

💬 리뷰 요구사항

  1. 알림 설정(ON/OFF)을 무시하고 강제 발송하는 테스트 전용 API라, 운영 배포 시 노출 범위(ROLE_ADMIN만 접근 가능)가 적절한지 검토 부탁드립니다.

@si-zero si-zero self-assigned this Aug 16, 2026
@si-zero si-zero added the ✨ Feat (기능 추가) 새로운 기능을 추가할 때 사용합니다. label Aug 16, 2026
@si-zero si-zero linked an issue Aug 16, 2026 that may be closed by this pull request
3 tasks
@si-zero
si-zero merged commit 70ccc90 into dev Aug 16, 2026
1 check passed
@si-zero
si-zero deleted the feat/#395 branch August 16, 2026 10:45
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: 관리자용 푸시 알림 테스트 API 추가

1 participant