-
Notifications
You must be signed in to change notification settings - Fork 0
fix: 발송/미발송 예약 메일 Local Standard Time(Lst)로 수정 #243
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
""" Walkthrough날짜 포맷팅 함수의 이름이 Changes
Possibly related issues
Poem
📜 Recent review detailsConfiguration used: CodeRabbit UI 📒 Files selected for processing (1)
🚧 Files skipped from review as they are similar to previous changes (1)
✨ Finishing Touches
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. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
m3k0813
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Gooood
There was a problem hiding this 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
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (2)
src/shared/lib/date.ts(1 hunks)src/widgets/dashboard/ui/email/SentMailCard.tsx(2 hunks)
🧰 Additional context used
🧬 Code Graph Analysis (1)
src/widgets/dashboard/ui/email/SentMailCard.tsx (1)
src/shared/lib/date.ts (1)
formatLst(47-57)
🔇 Additional comments (3)
src/shared/lib/date.ts (1)
47-48: 시간대 처리 로직 검증 필요: 브라우저 로컬 타임존 해석으로 변경되어 의도치 않은 시간 표시 가능현재 formatLst 함수(
src/shared/lib/date.ts)가 UTC → KST 수동 변환을 제거하고 new Date(utcString) 로 브라우저 로컬 타임존을 사용하도록 변경되었습니다.
그러나 예약일자(reservationDate)는“2025-05-01T14:00:00”형식(타임존 정보 없음)의 문자열로 정의되어 있어,
입력 문자열에 ‘Z’가 없거나 사용자가 KST가 아닌 환경에서 확인할 경우 예상과 다른 시간이 출력될 수 있습니다.확인 대상 위치:
- src/shared/lib/date.ts:
formatLst(utcString: string)구현부- src/widgets/dashboard/ui/email/SentMailCard.tsx:
formatLst(mail.reservationDate)호출부- src/features/dashboard/model/store/EmailStore.tsx:
reservationDate: string타입 정의- src/features/dashboard/model/email.ts:
EmailRequest·EmailResponse의reservationDate필드조치 제안:
- API 응답에 타임존 정보(예: “Z”) 포함 여부 확인
- 필요 시 formatLst 내부에서 명시적으로 UTC → KST 변환 적용하거나, 입력값에 타임존을 추가하도록 데이터 형식 통일
src/widgets/dashboard/ui/email/SentMailCard.tsx (2)
4-4: 함수명 변경에 따른 import 업데이트가 올바르게 적용되었습니다.
formatUtcToKst에서formatLst로의 함수명 변경이 올바르게 반영되었습니다.
41-41: 함수 호출 업데이트가 올바르게 적용되었습니다.함수명 변경에 따른 호출 부분이 올바르게 업데이트되었습니다. 함수 구현이 변경되어 시간 표시 방식이 달라질 수 있으므로, 실제 화면에서 예상한 로컬 시간이 올바르게 표시되는지 확인해보세요.
발송/미발송 예약 메일 로컬 시간으로 포맷팅
Summary by CodeRabbit