Skip to content
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

Merged
merged 2 commits into from
Feb 19, 2021
Merged

Conversation

youf-olivier
Copy link
Contributor

@youf-olivier youf-olivier commented Oct 23, 2020

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 :

  • Passage du composant à sa nouvelle version
  • Ajout de la doc de migration pour la V2 du toolkit car on a un breaking Change

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 :

  • Fusion des deux composants, avec possibilité de switch
  • Mise a jour du style en collaboration avec l'équipe UX

MIGRATION.md Outdated Show resolved Hide resolved
MIGRATION.md Outdated Show resolved Hide resolved
MIGRATION.md Outdated Show resolved Hide resolved
@youf-olivier
Copy link
Contributor Author

Ouais bon mon anglais est perfectible :D

MIGRATION.md Outdated Show resolved Hide resolved
MIGRATION.md Outdated Show resolved Hide resolved
packages/Form/Input/date/src/CustomDate.md Outdated Show resolved Hide resolved
packages/Form/Input/date/src/InputDate.md Outdated Show resolved Hide resolved
packages/Form/Input/date/src/InputDate.md Outdated Show resolved Hide resolved
@gcruchon
Copy link
Contributor

gcruchon commented Oct 24, 2020

I can't seem to make storybook work in FF and Safari

storybook - safari

@arnaudforaison
Copy link
Contributor

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 ?
https://developer.mozilla.org/fr/docs/Web/HTML/Element/Input/date

@youf-olivier
Copy link
Contributor Author

youf-olivier commented Oct 26, 2020

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 ?
https://developer.mozilla.org/fr/docs/Web/HTML/Element/Input/date

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.

@youf-olivier
Copy link
Contributor Author

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 new Date('02-25-2010') Bizarrement Chrome le prenait mais pas les autre, j'ai juste remplacé les - par des /

Enfin, j'ai mis RTL, et j'ai rendu l'async possible sur le tk.

Normalement c'est tout bon là !

MIGRATION.md Outdated Show resolved Hide resolved
MIGRATION.md Outdated Show resolved Hide resolved
MIGRATION.md Outdated Show resolved Hide resolved
MIGRATION.md Outdated Show resolved Hide resolved
MIGRATION.md Outdated Show resolved Hide resolved
packages/Form/Input/date/README.md Outdated Show resolved Hide resolved
packages/Form/Input/date/README.md Outdated Show resolved Hide resolved
packages/Form/Input/date/README.md Outdated Show resolved Hide resolved
packages/Form/Input/date/src/CustomDate.md Outdated Show resolved Hide resolved
packages/Form/Input/date/src/CustomDate.js Show resolved Hide resolved
xballoy
xballoy previously approved these changes Oct 27, 2020
babel.config.json Outdated Show resolved Hide resolved
packages/Form/Input/date/src/CustomDate.md Outdated Show resolved Hide resolved
@sonarcloud
Copy link

sonarcloud bot commented Oct 29, 2020

SonarCloud Quality Gate failed.

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities (and Security Hotspot 0 Security Hotspots to review)
Code Smell A 0 Code Smells

52.6% 52.6% Coverage
0.0% 0.0% Duplication

xballoy
xballoy previously approved these changes Oct 30, 2020
lerna.json Show resolved Hide resolved
Copy link
Contributor

@guillaume-chervet guillaume-chervet left a comment

Choose a reason for hiding this comment

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

Beau boulot 👍

arnaudforaison
arnaudforaison previously approved these changes Nov 5, 2020
xballoy
xballoy previously approved these changes Nov 24, 2020
@arnaudforaison arnaudforaison linked an issue Jan 15, 2021 that may be closed by this pull request
arnaudforaison
arnaudforaison previously approved these changes Jan 20, 2021
jforatier
jforatier previously approved these changes Feb 11, 2021
@arnaudforaison
Copy link
Contributor

Revue en point 'Revue de PR'

Olivier YOUF added 2 commits February 19, 2021 14:59
Initialized with the Date migration guide.
@arnaudforaison arnaudforaison dismissed stale reviews from jforatier, xballoy, and themself via 617eb45 February 19, 2021 14:37
@arnaudforaison arnaudforaison merged commit ae4d780 into master Feb 19, 2021
@delete-merged-branch delete-merged-branch bot deleted the feature/Date branch February 19, 2021 14:39
@sonarcloud
Copy link

sonarcloud bot commented Feb 19, 2021

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Feature request to migrate moment to dayjs
6 participants