Picker with Luxon config raises "Error: Invalid DateTime: unparsable: the input "2023-1-01" can't be parsed as format yyyy-MM-dd"
Chưa có ai nhận issue này.
Đá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
- 48/100
- Loại issue
- Lỗi
- Độ rõ ràng
- Khá rõ ràng
- Mức độ hoạt động
- Đình trệ
- Công nghệ
- react, typescript
- Lĩnh vực
- frontend
Hướng nghiên cứu
Bắt đầu với src/generate/luxon.ts và tái hiện trường hợp month-picker khi bật Luxon's Settings.throwOnInvalid, tập trung vào định dạng parsing được trích dẫn trong issue. Hoàn tất khi picker không còn phát sinh lỗi với cả đầu vào tháng có padding và không có padding, đồng thời giữ nguyên hành vi ngày hiện có.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
rc-picker version: 3.8.1
antd version: 5.6.0
luxon version: 3.3.0
When we set the luxon config to Settings.throwOnInvalid = true we get an error when we open up the antd date range picker (using "month" picker)
I don't deeply understand the rc-picker code, but a few things stuck out to me:
the luxon code is parsing using a "yyyy-MM-dd" format while dayjs seems to allow for both padded and non-padded month ['YYYY-M-DD', 'YYYY-MM-DD']
This doesn't really cause issues if we ignore the error, so our workaround is to change that throwOnInvalid setting, but we would strongly prefer for parsing errors to throw. This may be an issue with antd's picker and could raise the issue there if you think that's better, but I do think the config generation for different libraries should have the same behavior. If dayjs accepts that format, then it seems like luxon also should accept it. I did a quick test and I believe "yyyy-M-dd" will work for both 2023-1-01 and 2023-01-01 cases, so that is potentially a simple fix if this doesn't have other side effects.
- Ngôn ngữ chính
- TypeScript
- Star
- 304
- Fork
- 349
- Merge trung bình
- 43 phút
- Pull request đã merge (30 ngày)
- 1
Chuẩn bị môi trường
Dự án này không cung cấp dev container, Dockerfile hay hướng dẫn đóng góp, nên bạn cần tự thiết lập môi trường: hãy bắt đầu từ README và xem hướng dẫn đóng góp lần đầu của chúng tôi để biết các bước chung.
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- 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.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của react-component/picker
-
Disabled date entered manually is not reverted on blurCó thể đã có người làm @Ye-YiChen đã nhận 4 ngày trước. Đang mở
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 68/100
react-component/picker#1018 · 2 bình luận ·
-
enter date manually in format YY prevent the user to add YYYYCó thể đã có người làm @nrps9909 đã nhận 50 ngày trước. Đang mở
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 48/100
react-component/picker#956 · 1 bình luận · 1 reaction ·
-
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 30/100
react-component/picker#955 · 1 bình luận · 1 reaction ·
-
[Bug / DX]: Deep imports for generateConfig are incompatible with pnpm and break module resolutionCó thể đã có người làm @DanielChutkowskiJMMJ đã nhận 332 ngày trước. Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 58/100
react-component/picker#949 ·
-
DatePicker does not trigger onChange when manually clearing input textCó thể đã có người làm @nrps9909 đã nhận 50 ngày trước. Đang mở
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 55/100
react-component/picker#946 ·
Tất cả issue của react-component/picker
Issue tương tự
-
`cf complete zsh >> .zshrc` fails on vanilla zshCó thể làm lại được Pull request cho issue này đã bị đóng mà không được merge. Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
cloudflare/cf#340 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
-
Rename Madson cameraĐang mở
Độ khó 2/5 Dưới một giờ Mức phù hợp với người mới 75/100
StoDevX/AAO-React-Native#8785 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Broken links in the docsĐang mở
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 75/100
-
bnv-manager-v2: accept_invite creates the Mailcow domain admin before the commitCó thể đã có người làm @myOmikron đã nhận hôm nay. Đang mởbug ready-for-agent
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 82/100