Skip to content

#212 [Fix] 알림 노출 기준(user_id) 정리 및 관리자 공지 저장 플로우 검증 보강#214

Merged
jucheonsu merged 1 commit intodevfrom
fix/#212
Apr 24, 2026
Merged

#212 [Fix] 알림 노출 기준(user_id) 정리 및 관리자 공지 저장 플로우 검증 보강#214
jucheonsu merged 1 commit intodevfrom
fix/#212

Conversation

@jucheonsu
Copy link
Copy Markdown
Member

@jucheonsu jucheonsu commented Apr 24, 2026

#️⃣ 연관된 이슈

📝 작업 내용

♻️ Refactor

내용 파일
사용자 알림 가시성 판별 기준을 카테고리 분기에서 user_id 기준(전체:null / 개인:not null)으로 정리 NotificationRepository.java, NotificationService.java

🐛 Fix

내용 파일
전체 공지 성격의 CONTENT 알림이 사용자 목록/상세에서 누락되지 않도록 조회/읽음 처리 로직 수정 NotificationService.java, NotificationRepository.java
관리자 공지 페이지 저장 플로우 검증(페이지 렌더링 요소 + 등록 + 관리자/사용자 조회) 통합 테스트 추가 AdminNoticeIntegrationTest.java
CONTENT 브로드캐스트 알림 사용자 노출 회귀 테스트 추가 NotificationServiceTest.java

📌 공유 사항

  1. 관리자 공지 등록 후 user_id == null 레코드는 CONTENT/NOTICE/EVENT 공통으로 사용자 조회 대상입니다.
  2. 개인 대상 알림(user_id != null)은 기존처럼 해당 사용자만 조회/읽음 처리됩니다.

✅ 체크리스트

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

📸 스크린샷

  • 테스트 실행: ./gradlew test --tests "com.swyp.picke.domain.admin.controller.AdminNoticeIntegrationTest" --tests "com.swyp.picke.domain.admin.controller.AdminContentCreationIntegrationTest" --tests "com.swyp.picke.domain.scenario.service.ScenarioServiceImplTest" --tests "com.swyp.picke.domain.admin.controller.AdminScenarioPublishFlowIntegrationTest"
  • 결과: BUILD SUCCESSFUL

💬 리뷰 요구사항

  1. CONTENT를 포함한 전체 공지 알림을 user_id 기준으로 통일한 정책이 운영 의도와 일치하는지 확인 부탁드립니다.
  2. 관리자 공지 페이지 저장 흐름 검증 수준(페이지 요소/저장/조회)이 추가로 보강되어야 할 지점이 있다면 의견 부탁드립니다.

@jucheonsu jucheonsu self-assigned this Apr 24, 2026
@jucheonsu jucheonsu added the 🐛 Fix (버그 수정) 버그를 수정할 때 사용합니다. label Apr 24, 2026
@jucheonsu jucheonsu merged commit 1070246 into dev Apr 24, 2026
@jucheonsu jucheonsu deleted the fix/#212 branch April 24, 2026 14:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🐛 Fix (버그 수정) 버그를 수정할 때 사용합니다.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

🐛 Fix: 관리자 공지 등록 후 사용자 알림 조회 누락 및 페이지 저장 흐름 검증 보강

1 participant