Skip to content

fix: 투표 종료 알림 body 정리 및 투표 기간 분 단위 옵션 추가#234

Merged
Junhyukkkk merged 3 commits into
developfrom
fix/notification-body-remove-decoration
Jun 12, 2026
Merged

fix: 투표 종료 알림 body 정리 및 투표 기간 분 단위 옵션 추가#234
Junhyukkkk merged 3 commits into
developfrom
fix/notification-body-remove-decoration

Conversation

@Junhyukkkk

@Junhyukkkk Junhyukkkk commented Jun 12, 2026

Copy link
Copy Markdown
Member

📌 관련 이슈

  • closes #

🔍 작업 내용

  • 프론트 UI 시안 반영 요청에 따라 투표 종료 알림의 body 값을 순수 투표 제목만 내려주도록 수정했습니다.
  • 투표 생성 시 선택할 수 있는 기간 옵션에 분 단위(7/10/15분)를 추가했습니다.

📝 변경 사항

  • 알림 body 형식 변경
    • AS-IS: "[투표 제목] 결과 보러가기"
    • TO-BE: "투표 제목" (대괄호 및 ' 결과 보러가기' 문구 제거)
  • 변경된 응답 형식에 맞춰 알림 컨트롤러 테스트 더미 데이터 정리
  • VoteDuration에 분 단위 옵션 추가 (MINUTES_7, MINUTES_10, MINUTES_15)

💬 리뷰어에게

  • 알림 title("투표 결과가 공개됐어요")은 별개 필드라 변경 범위에서 제외했습니다. 함께 조정이 필요한지 확인 부탁드립니다.
  • 분 단위 옵션이 짧은 편이라, 스케줄러/알림 발송 주기 등 기간 의존 로직에 영향이 없는지 봐주시면 좋겠습니다.

Summary by CodeRabbit

릴리스 노트

  • 새로운 기능
    • 더 짧은 투표 기간 옵션 추가 (7분, 10분, 15분)
    • 투표 종료 알림 메시지 형식 간소화

@Junhyukkkk Junhyukkkk self-assigned this Jun 12, 2026
@github-actions github-actions Bot added the bug label Jun 12, 2026
@coderabbitai

coderabbitai Bot commented Jun 12, 2026

Copy link
Copy Markdown

Review Change Stack

Caution

Review failed

Pull request was closed or merged during review

Walkthrough

PR는 투표 지속 시간 옵션을 확장하고 투표 종료 알림 메시지 형식을 단순화합니다. 분 단위 투표 기간이 추가되며, 알림 본문이 제목만 직접 사용하도록 변경되고 관련 테스트가 이를 반영합니다.

Changes

투표 기간 및 알림 업데이트

Layer / File(s) Summary
투표 기간 열거형 확장
src/main/java/com/ject/vs/vote/domain/VoteDuration.java
VoteDuration 열거형에 MINUTES_7, MINUTES_10, MINUTES_15 상수가 추가되어 분 단위의 짧은 투표 주기를 지원합니다.
알림 메시지 형식 단순화
src/main/java/com/ject/vs/notification/domain/Notification.java, src/unitTest/java/com/ject/vs/notification/adapter/web/NotificationControllerTest.java
Notification.ofVoteEnded()body 필드가 "[제목] 결과 보러가기" 형식에서 투표 제목만 직접 사용하도록 변경되며, 테스트 기댓값이 이에 맞춰 업데이트됩니다.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Suggested labels

bug, feature

Suggested reviewers

  • tlarbals824
  • KII1ua

Poem

🐰 분 단위 시간을 더하고,
알림 메시지는 깔끔하게—
투표는 유연해지고,
사용자 경험은 더 명확해진다.
작고 소중한 변화들이 모여,
서비스를 한 단계 나아가게 한다. 🎯

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed PR 제목은 변경 사항의 두 가지 주요 내용을 정확하게 요약하고 있습니다: 투표 종료 알림 body 정리 및 투표 기간 분 단위 옵션 추가.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/notification-body-remove-decoration

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions

Copy link
Copy Markdown

빌드 성공
배포 준비 완료!

@Junhyukkkk Junhyukkkk merged commit b34abb7 into develop Jun 12, 2026
4 of 5 checks passed
@Junhyukkkk Junhyukkkk deleted the fix/notification-body-remove-decoration branch June 12, 2026 14:50
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.

1 participant