Picker with Luxon config raises "Error: Invalid DateTime: unparsable: the input "2023-1-01" can't be parsed as format yyyy-MM-dd"
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 2/5
- Temps estimé
- 1-3 heures
- Accessibilité débutants
- 48/100
- Type d'issue
- Bug
- Clarté
- Plutôt claire
- Activité
- À l'abandon
- Stack technique
- react, typescript
- Domaine
- frontend
Piste de recherche
Commencez par src/generate/luxon.ts et reproduisez le cas du month-picker avec Luxon's Settings.throwOnInvalid activé, en vous concentrant sur le format de parsing cité dans l'issue. C'est terminé lorsque le picker ne lève plus d'erreur pour les entrées de mois avec ou sans padding, tout en préservant le comportement actuel des dates.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
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.
- Langage dominant
- TypeScript
- Étoiles
- 304
- Forks
- 349
- Merge moyen
- 43 min
- PR mergées (30 j)
- 1
Préparer son environnement
Ce projet ne fournit ni conteneur de développement, ni Dockerfile, ni guide de contribution : l'installation est à votre charge. Commencez par son README, et consultez notre guide de la première contribution pour les étapes générales.
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de react-component/picker
-
Disabled date entered manually is not reverted on blurPeut-être pris @Ye-YiChen l’a pris il y a 8 jours. Ouverte
Difficulté 3/5 1-2 jours Accessibilité débutants 68/100
react-component/picker#1018 · 2 commentaires ·
-
enter date manually in format YY prevent the user to add YYYYPeut-être pris @nrps9909 l’a pris il y a 54 jours. Ouverte
Difficulté 3/5 1-2 jours Accessibilité débutants 48/100
react-component/picker#956 · 1 commentaire · 1 réaction ·
-
Difficulté 4/5 3-5 jours Accessibilité débutants 30/100
react-component/picker#955 · 1 commentaire · 1 réaction ·
-
[Bug / DX]: Deep imports for generateConfig are incompatible with pnpm and break module resolutionPeut-être pris @DanielChutkowskiJMMJ l’a pris il y a 336 jours. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 58/100
react-component/picker#949 ·
-
DatePicker does not trigger onChange when manually clearing input textPeut-être pris @nrps9909 l’a pris il y a 54 jours. Ouverte
Difficulté 3/5 1-2 jours Accessibilité débutants 55/100
react-component/picker#946 ·
Toutes les issues de react-component/picker
Issues similaires
-
批量图谱关联的备份应排除Obsidian索引Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 70/100
logancyang/obsidian-copilot#3753 ·
Les mainteneurs répondent en général sous 1 jour
-
area: presentation severity: important
Difficulté 2/5 1-3 heures Accessibilité débutants 85/100
voithos-labs/aragonite#763 ·
Les mainteneurs répondent en général sous 2 jours
-
fix(cursor): align Grok 4.7 context window to 256k/500k and drop unsupported max effort tierOuverte
Difficulté 2/5 1-3 heures Accessibilité débutants 74/100
diegosouzapw/OmniRoute#17386 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
Les mainteneurs répondent en général sous 1 jour
-
💡 [Feature] Add an option to hide the empty screen background illustration (Distraction-Free)Ouverte
Difficulté 2/5 Une demi-journée Accessibilité débutants 72/100
super-productivity/super-productivity#10982 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour