Hacktoberfest 2026: những issue maintainer đã đánh dấu cho tháng Mười, đang mở và phù hợp người mới. Xem issue Hacktoberfest

Timestamp converter still turns "02/30/2024" into March 1

Đã đóng Phù hợp với người mới
  • Có gì không ổn?
#345 1 bình luận 0 reaction 0 người được giao Xem trên GitHub

Maintainer thường phản hồi trong vòng 1 ngày

@bfs2021 đang làm issue này rồi.

Từ ngày 25/10/2026.

  • #346 của @bfs2021 — đang mở

Đánh giá

Độ khó
2/5
Thời gian dự kiến
1-3 giờ
Mức phù hợp với người mới
88/100
Loại issue
Lỗi
Độ rõ ràng
Đặc tả rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
typescript
Lĩnh vực
frontend, testing

Hướng nghiên cứu

Bắt đầu với extractCalendarDay và mẫu ngày tháng dạng số trong lib/time-utils.ts, sau đó đọc các trường hợp liên quan trong lib/__tests__/time-utils.test.ts. Bổ sung kiểm thử cho cả 02/30/2024 và 2/30/2024, đồng thời xác nhận 02/29/2024 và 12/31/2024 vẫn được chuyển đổi. Hoàn tất khi các ngày không hợp lệ theo định dạng Mỹ bị từ chối và các lệnh lint, test, build được nêu đều chạy thành công.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

bug good first issue hacktoberfest

#340 made the timestamp converter reject impossible days in 2024-02-30, 2024/02/30, February 30, 2024 and 30 February 2024. US-style dates still slip through: type 02/30/2024 (month/day/year) into Date to epoch and it converts it to March 1, 2024. extractCalendarDay (lib/time-utils.ts, line 143) only reads numeric dates that start with the year (line 144), so MM/DD/YYYY goes straight to Date.parse, which rolls the day over.

What to do

  • Read MM/DD/YYYY (and M/D/YYYY) in extractCalendarDay the way Date.parse reads it, month first, so the day can be checked against that month's length.

Where to look

  • lib/time-utils.ts: extractCalendarDay (line 143) and its numeric pattern (line 144)
  • lib/__tests__/time-utils.test.ts

Acceptance criteria

  • 02/30/2024 and 2/30/2024 are rejected like 2024-02-30
  • 02/29/2024 and 12/31/2024 still convert
  • Tests that fail on the current code
  • npm run lint, npm run test and npm run build pass

Want to work on this? Comment below and I'll assign it to you. Please take one issue at a time, branch off develop, and open your PR against develop with Closes #345 in the description.

Ngôn ngữ chính
TypeScript
Star
1
Fork
9
Merge trung bình
7 giờ 17 phút
Pull request đã merge (30 ngày)
158

Chuẩn bị môi trường

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của codercops/toolbelt

Tất cả issue của codercops/toolbelt

Issue tương tự

Thêm issue về TypeScript

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.