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

fix(slash-react-date): correct issue with formatDateValue #741

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

ashnarrainen
Copy link

Fix issue where "Date" component was not receiving the DateValue in the correct Format. Hence getMonth() was returning an exception:

image

@MartinWeb
Copy link
Contributor

mettre fix(slash): correct issue with formatDateValue en message de commit

Copy link
Contributor

@GuillaumeKESTEMAN GuillaumeKESTEMAN left a comment

Choose a reason for hiding this comment

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

J'approuve car c'est un fix qui est urgent mais je me demande si avec date-fns on ne peut pas éviter d'avoir à gérer tout ça ?

Edit: je viens de voir qu'il était importé par concurrently et pas manuellement donc je n'ai rien dit 😆

@ashnarrainen ashnarrainen force-pushed the fixlease/datepicker-correction branch from 1a87820 to e7ebed9 Compare January 15, 2025 08:24
Comment on lines +12 to +15
const formattedDateValue = new globalThis.Date(dateValue);
const monthFormatted = `0${formattedDateValue.getMonth() + 1}`.slice(-2);
const dayFormatted = `0${formattedDateValue.getDate()}`.slice(-2);
return `${formattedDateValue.getFullYear()}-${monthFormatted}-${dayFormatted}`;
Copy link
Contributor

Choose a reason for hiding this comment

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

Il y a beaucoup plus simple

Suggested change
const formattedDateValue = new globalThis.Date(dateValue);
const monthFormatted = `0${formattedDateValue.getMonth() + 1}`.slice(-2);
const dayFormatted = `0${formattedDateValue.getDate()}`.slice(-2);
return `${formattedDateValue.getFullYear()}-${monthFormatted}-${dayFormatted}`;
return dateValue.toISOString().slice(0, 10);

@ashnarrainen
Copy link
Author

@GuillaumeKESTEMAN aussi la raison pourquoi j'ai dû utiliser le GlobalThis c'est parce que le component Date a le même nom que le type Date.

Je suis plutot chaud de renommer le composant de Date > DatePicker pour entre plus parlant sur ce que le composant fait et aussi pas clash avec le typeDat. Dans une prochaine version majeure biensure :)

@GuillaumeKESTEMAN
Copy link
Contributor

@GuillaumeKESTEMAN aussi la raison pourquoi j'ai dû utiliser le GlobalThis c'est parce que le component Date a le même nom que le type Date.

Je suis plutot chaud de renommer le composant de Date > DatePicker pour entre plus parlant sur ce que le composant fait et aussi pas clash avec le typeDat. Dans une prochaine version majeure biensure :)

Je suis d'accord, ce serait mieux mais attention cela fera un breaking change ⚠

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

Successfully merging this pull request may close these issues.

5 participants