Skip to content

Comments

한국 시간 기준으로 스터디 시작일 유효성 검사 로직 수정#374

Merged
HA-SEUNG-JEONG merged 1 commit intodevelopfrom
fix/study
Feb 4, 2026
Merged

한국 시간 기준으로 스터디 시작일 유효성 검사 로직 수정#374
HA-SEUNG-JEONG merged 1 commit intodevelopfrom
fix/study

Conversation

@HA-SEUNG-JEONG
Copy link
Contributor

@HA-SEUNG-JEONG HA-SEUNG-JEONG commented Feb 4, 2026

🌱 연관된 이슈

☘️ 작업 내용

🍀 참고사항

스크린샷 (선택)

Summary by CodeRabbit

릴리스 노트

  • 버그 수정
    • 스터디 그룹 생성 시 시작 날짜 검증 로직을 개선했습니다. 이제 한국 표준시(KST) 기준으로 정확하게 계산되어, 사용자의 로컬 시간대 설정에 관계없이 일관된 날짜 검증이 이루어집니다.

@HA-SEUNG-JEONG HA-SEUNG-JEONG self-assigned this Feb 4, 2026
@vercel
Copy link

vercel bot commented Feb 4, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
study-platform-client-dev Ready Ready Preview, Comment Feb 4, 2026 7:53am

@coderabbitai
Copy link

coderabbitai bot commented Feb 4, 2026

📝 Walkthrough

개요

그룹 스터디 폼 스키마의 시작 날짜 검증 로직이 한국 시간대 기준으로 변경되었습니다. 기존 Date 객체 비교에서 getKoreaDate()를 활용한 YYYY-MM-DD 문자열 비교 방식으로 전환되어 한국 로컬 시간 기준 검증을 보장합니다.

변경사항

Cohort / File(s) Summary
날짜 검증 로직 개선
src/features/study/group/model/group-study-form.schema.ts
getKoreaDate() 임포트 추가 및 시작 날짜 검증 로직을 한국 시간대 인식 방식으로 리팩토링. Date 객체 비교에서 YYYY-MM-DD 문자열 비교로 변경하여 시작 날짜가 한국 로컬 기준으로 최소한 내일 이후여야 함을 보장. 날짜 포매팅을 위한 toYmd 헬퍼 함수 추가.

예상 코드 리뷰 시간

🎯 2 (Simple) | ⏱️ ~10분

시인의 작품

🐰 한국 시간으로 내일을 맞추다니,
날짜 문자열 비교로 깔끔해졌네.
getKoreaDate()의 마법으로,
검증도 정확하고 참 좋아라! 🌙✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Title check ✅ Passed PR 제목이 변경사항의 주요 내용을 명확하게 설명하고 있습니다. 한국 시간 기준 스터디 시작일 유효성 검사 로직 수정이라는 핵심 변경사항을 정확하게 반영하고 있습니다.

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

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/study

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

@HA-SEUNG-JEONG HA-SEUNG-JEONG changed the title 코드 래빗 코멘트 반영 한국 시간 기준으로 스터디 시작일 유효성 검사 로직 수정 Feb 4, 2026
@HA-SEUNG-JEONG HA-SEUNG-JEONG merged commit d72a52e into develop Feb 4, 2026
10 checks passed
@HA-SEUNG-JEONG HA-SEUNG-JEONG deleted the fix/study branch February 4, 2026 07:58
@coderabbitai coderabbitai bot mentioned this pull request Feb 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant