Skip to content

feat: add isTimeZone() validator decorator#213

Merged
dami0806 merged 5 commits intodevelopfrom
CARGO-397
Apr 14, 2026
Merged

feat: add isTimeZone() validator decorator#213
dami0806 merged 5 commits intodevelopfrom
CARGO-397

Conversation

@dami0806
Copy link
Copy Markdown
Contributor

No description provided.

@dami0806 dami0806 self-assigned this Apr 11, 2026
@auto-assign auto-assign Bot requested review from laggu, sezeme and yuchem2 April 11, 2026 03:36
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

이번 풀 리퀘스트는 유효한 IANA 타임존 식별자인지 검증하는 @IsTimeZone 데코레이터를 추가합니다. 해당 기능은 내장 Intl API를 사용하여 구현되었으며, 관련 문서 업데이트, 예제 코드 추가 및 단위 테스트 작성이 포함되었습니다. 현재 리뷰어의 피드백이 없으므로 별도로 제공할 의견은 없습니다.

yuchem2
yuchem2 previously approved these changes Apr 13, 2026
@laggu
Copy link
Copy Markdown
Member

laggu commented Apr 13, 2026

conflict 났어요

@yuchem2
Copy link
Copy Markdown
Member

yuchem2 commented Apr 13, 2026

일본어 버전 추가되서 일본어 버전도 추가해주시면 좋을 것 같아요!

@dami0806
Copy link
Copy Markdown
Contributor Author

conflict 해결과 일본어 추가 완료 했습니다!

@dami0806 dami0806 merged commit 343a11b into develop Apr 14, 2026
3 checks passed
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.

3 participants