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

Meilleur affichage des dates relatives #650

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

azerty29242
Copy link

Checklist d'avant pull request

  • Vous avez testé de build le projet avec vos modifications et ce build a réussi
  • Vous respectez les conventions de codage et de nommage du projet
  • Vous utilisez la tabulation pour l'indentation afin de maintenir un code lisible
  • Cette pull request n'est pas un duplicata d'une autre
  • Cette pull request est prête à être revue (review) et fusionnée (merge)
  • Il n'y a pas de TODO (aka des annotations pour du code manquant) dans vos modifications
  • Il n'y a pas d'erreurs de langue dans votre code (grammaire, vocabulaire, conjugaison, orthographe)
  • Les détails des changements ont été décrits ci-dessous
  • Cette pull-request n'est pas une "breaking-change" (des modifications qui vont entraîner la modification du fonctionnement de certaines fonctionnalités déjà existantes)

Changelogs proposés

Papillon affiche maintenant Hier, Avant-hier ou Il y a ... jours à la place de Dans -... jours. Ce n'est pas une grande modification mais c'est à voir comme un petite amélioration de l'expérience globale de l'utilisateur.

Informations supplémentaires

Il est aussi possible d'utiliser Intl.RelativeTimeFormat s'il faut.

Copy link
Contributor

@raphckrman raphckrman left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@Kgeek33 Kgeek33 left a comment

Choose a reason for hiding this comment

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

Correct, mais je recommanderai d'utiliser un switch/case pour un code plus simple

@azerty29242
Copy link
Author

Correct, mais je recommanderai d'utiliser un switch/case pour un code plus simple

C'est bon, c'est fait

Copy link
Contributor

@Gabriel29306 Gabriel29306 left a comment

Choose a reason for hiding this comment

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

Oui

Copy link
Contributor

@Kgeek33 Kgeek33 left a comment

Choose a reason for hiding this comment

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

Parfait LGTM!

@azerty29242
Copy link
Author

azerty29242 commented Jan 18, 2025

Je viens de modifier le code pour que le programme puisse parler en mois et en années pour les dates lointaines. J'ai fait un code pour tester le programme avec pleins de dates et voici le résultat :

18/01/2023 - Il y a 2 ans
19/01/2023 - Il y a 1 an
18/01/2024 - Il y a 1 an
19/01/2024 - Il y a 11 mois
18/12/2024 - Il y a 1 mois
19/12/2024 - Il y a 30 jours
15/01/2025 - Il y a 3 jours
16/01/2025 - Avant-hier
17/01/2025 - Hier
18/01/2025 - Aujourd'hui
19/01/2025 - Demain
20/01/2025 - Après-demain
21/01/2025 - Dans 3 jours
17/02/2025 - Dans 30 jours
18/02/2025 - Dans 1 mois
17/01/2026 - Dans 11 mois
18/01/2026 - Dans 1 an
17/01/2027 - Dans 1 an
18/01/2027 - Dans 2 ans

Je ne suis pas chez moi et ne peut donc pas tester le programme avec Papillon. Est-ce que quelqu'un pourrait faire le test ? Si quelqu'un a des idées pour rendre le code meilleur, je suis preneur.

Copy link
Contributor

@Gabriel29306 Gabriel29306 left a comment

Choose a reason for hiding this comment

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

Juste un peu de simplification, sinon c'est nickel !

src/utils/format/DateHelper.ts Show resolved Hide resolved
@Gabriel29306
Copy link
Contributor

Après redemande moi

Copy link
Contributor

@Gabriel29306 Gabriel29306 left a comment

Choose a reason for hiding this comment

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

LGTM

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