Skip to content

[FIX] 현재포함 미래로 변경#204

Merged
PBEM22 merged 2 commits intodevelopfrom
fix/#203/FutureOrPresent
Jan 23, 2026
Merged

[FIX] 현재포함 미래로 변경#204
PBEM22 merged 2 commits intodevelopfrom
fix/#203/FutureOrPresent

Conversation

@PBEM22
Copy link
Copy Markdown
Collaborator

@PBEM22 PBEM22 commented Jan 23, 2026

📌 관련 이슈

✨ 변경 사항

  • 모집 마감일 현재포함 미래로 변경

📸 테스트 증명 (필수)

📚 리뷰어 참고 사항

✅ 체크리스트

  • 브랜치 전략(git flow)을 따랐나요? (develop -> feat/...)
  • 로컬에서 빌드 및 실행이 정상적으로 되나요?
  • 불필요한 주석(TODO 등)이나 더미 코드는 제거했나요?
  • 컨벤션(커밋 메시지, 코드 스타일)을 지켰나요?

Summary by CodeRabbit

  • 버그 수정
    • 그룹 구매 마감일 설정 시 오늘(현재 날짜)을 유효한 값으로 허용하도록 변경되었습니다. 이전에는 오직 미래 날짜만 허용되어 마감일을 당일로 설정할 수 없던 문제가 해결되었습니다.

✏️ Tip: You can customize this high-level summary in your review settings.

@PBEM22 PBEM22 requested a review from 88guri January 23, 2026 13:28
@PBEM22 PBEM22 self-assigned this Jan 23, 2026
@PBEM22 PBEM22 added 🚨 Fix 버그 수정 🔥 P0 당장 개발해야 하는 핵심 기능 🐴 채륜 막시무스 labels Jan 23, 2026
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Jan 23, 2026

📝 Walkthrough

Walkthrough

GroupBuyCreateRequest의 deadline 필드 유효성 검사 애너테이션을 @Future에서 @FutureOrPresent로 변경하여, 오늘 날짜를 포함한 미래 날짜를 유효하도록 수정했습니다.

Changes

Cohort / File(s) 변경 사항
Validation Constraint Update
src/main/java/org/sopt/poti/domain/groupbuy/dto/request/GroupBuyCreateRequest.java
deadline 필드의 유효성 검사 제약을 @Future@FutureOrPresent로 변경. 관련 import도 jakarta.validation.constraints.Futurejakarta.validation.constraints.FutureOrPresent로 교체.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed PR 제목이 변경사항의 핵심을 정확히 반영하고 있으며, 마감일 검증 조건을 '@future'에서 '@FutureOrPresent'로 변경한 내용을 간결하게 표현하고 있습니다.
Linked Issues check ✅ Passed PR의 코드 변경사항이 #203 이슈의 요구사항을 완전히 충족하고 있습니다. '@FutureOrPresent' 적용으로 오늘을 포함한 미래 날짜를 유효한 마감일로 인정하는 요구사항이 구현되었습니다.
Out of Scope Changes check ✅ Passed 모든 변경사항이 관련 이슈 #203의 범위 내에 있으며, GroupBuyCreateRequest의 deadline 필드 검증 규칙 변경만 포함되어 있습니다.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

✨ Finishing touches
  • 📝 Generate docstrings

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.

Copy link
Copy Markdown

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Fix all issues with AI agents
In
`@src/main/java/org/sopt/poti/domain/groupbuy/dto/request/GroupBuyCreateRequest.java`:
- Around line 22-24: The validation annotation on the deadline field (LocalDate
deadline) uses `@FutureOrPresent` but the message says it must be "future",
causing a mismatch; update the validation message to reflect `@FutureOrPresent`
(e.g., change the message to indicate "오늘 또는 미래여야 합니다" or similar) in
GroupBuyCreateRequest so the annotation and error text align.

Copy link
Copy Markdown
Collaborator

@88guri 88guri left a comment

Choose a reason for hiding this comment

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

클라와 빠른 소통 감사합니다 ~

@PBEM22 PBEM22 merged commit 87f3125 into develop Jan 23, 2026
2 checks passed
@PBEM22 PBEM22 deleted the fix/#203/FutureOrPresent branch January 23, 2026 14:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🐴 채륜 막시무스 🚨 Fix 버그 수정 🔥 P0 당장 개발해야 하는 핵심 기능

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[FIX] 모집 마감일 오늘포함 미래로 변경

2 participants