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

Renomme et change le comportement du templatetag humane_time #6521

Merged

Conversation

philippemilink
Copy link
Member

Le templatetag humane_time faisait deux choses : il convertissait un timestamp en un objet datetime et appliquait directement dessus le templatetag format_date.

Plusieurs choses n'allaient pas :

  • le nom n'est pas assez explicit
  • si on a un timestamp, on obligé d'appliquer le templatetag format_date, aucun autre, sans possibilité de préciser d'éventuelles options
  • ce templatetag faisait deux choses, c'est-à-dire une de trop

J'ai transformé ce templatetag en date_from_timestamp : il ne fait que convertir un timestamp en objet datetime, ce qui nous donne ensuite toute liberté pour appliquer n'importe quel templatetag de temps ensuite.

(j'en ai besoin pour la PR pour la recherche)

Contrôle qualité

  • Générer et relire la documentation
  • Se connecter en tant qu'un membre qui a publié un tutoriel, aller sur la page de l'historique du tutoriel : les dates des commits doivent être correctement affichées, et en cliquant sur Activer pour activer la bêta, la date dans le message de la modale doit aussi être bien affichée.

@philippemilink philippemilink added this to En développement in Suivi des PR via automation Jul 26, 2023
@philippemilink philippemilink moved this from En développement to En attente de QA in Suivi des PR Jul 26, 2023
@coveralls
Copy link

coveralls commented Jul 26, 2023

Coverage Status

coverage: 88.396%. remained the same when pulling 62fd3b0 on philippemilink:rename-humane_time-templatetag into 44468ca on zestedesavoir:dev.

Copy link
Member

@Situphen Situphen left a comment

Choose a reason for hiding this comment

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

QA OK Rien à redire !

Suivi des PR automation moved this from En attente de QA to Fusionnable après rebase Jul 26, 2023
@Situphen Situphen merged commit 2017176 into zestedesavoir:dev Jul 26, 2023
12 checks passed
Suivi des PR automation moved this from Fusionnable après rebase to Fusionnée Jul 26, 2023
@philippemilink philippemilink deleted the rename-humane_time-templatetag branch July 26, 2023 17:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Suivi des PR
  
Fusionnée
Development

Successfully merging this pull request may close these issues.

None yet

3 participants