-
Notifications
You must be signed in to change notification settings - Fork 62
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
feat(Date): bump react-datepicker to 3.X #767
Conversation
b07046e
to
af1b54d
Compare
Ouais bon mon anglais est perfectible :D |
Question ouverte. Pourquoi passe ton par une lib avec des choses plus ou moins complexes plutôt que de se baser sur le standard qui prend la local du navigateur par ex ? |
Je te propose d'en parler dans l'issue #712 . Ici l'idée est de préparer la suite pour le champs Date Picker. J'ai fait justement une petite PR pour sortir de moment et mettre a jour le composant. Le temps de discuter de la suite avec les UX Mais pour répondre a ta question, de mémoire le composant natif est pas customisable visuellement. |
af1b54d
to
de67509
Compare
Hello @guillaume-chervet J'ai repush une version avec le chargement dynamique. Ca semble bien fonctionner. On a tester ça avec @arnaudforaison et @xballoy Pour ton erreur @gcruchon c'était bidon, j'avais initié dans la story la date avec un Enfin, j'ai mis RTL, et j'ai rendu l'async possible sur le tk. Normalement c'est tout bon là ! |
de67509
to
2ba9b16
Compare
80c3df5
to
c1181eb
Compare
2cd3aba
to
5b67649
Compare
5b67649
to
b5fb870
Compare
SonarCloud Quality Gate failed.
|
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.
Beau boulot 👍
9e42611
9af3bb5
to
9e42611
Compare
Revue en point 'Revue de PR'
|
…emove moment and uses JS Date
Initialized with the Date migration guide.
617eb45
9e42611
to
617eb45
Compare
SonarCloud Quality Gate failed. |
Passage de React Date picker à 3.X sur le composant date.
Ca créé un breaking car moment saute à la faveur de DateFns en dépendance qui renvoie un objet Date.
Il y a donc deux commits :
J'aurai besoin par contre que l'on discute de la façon de passer la locale. J'ai créé 5 locales de bases, pour pas tout importer.
Mais je sais pas si cette façon est idéale.
EDIT :
C'est une premier PR. Je dois ajouter d'autre travaux à la suite de celle ci :