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

Améliore l'affichage de l'estimation du temps de lecture #6401

Merged
merged 1 commit into from
Oct 12, 2022

Conversation

Arnaud-D
Copy link
Contributor

Reprise de #5998.

Fix #5780.

J'ai changé un peu par rapport à ce qu'avait fait Toxic à l'époque.

On a une précision changeante, de moins en moins fine à mesure que les valeurs de temps devienne grande. On passe de la minute, à 5 min, à 10 min, puis 15 min puis 30 min, puis 60 min. Comme ça, la valeur a une précision adaptée à la longueur : pas besoin d'être précis à 15 min sur une publication qui met plusieurs dizaines d'heures à être lue.

Le filtre est flexible pour adapter facilement les valeurs de précision ou les bornes des intervalles.

J'en ai profité pour refactoriser un peu (changement de nom) et me suis essayé aux annotations de type.

Contrôle qualité

  • Relire le code et les tests pour juger de la pertinence des troncatures.
  • Importer l'archive d'un très gros contenu (par exemple le tuto arduino) et voir que le temps est affiché uniquement en heures.
  • Importer l'archive d'un article plus court et constater le temps affiché en minutes.
  • Possibilité de faire ça sur des choses intermédiaires, mais c'est plus dur de bien viser sur la durée. :D

@Arnaud-D Arnaud-D added C-Back Concerne le back-end Django hacktoberfest-accepted Pull request approuvée pour le Hacktoberfest labels Oct 11, 2022
@coveralls
Copy link

coveralls commented Oct 11, 2022

Coverage Status

Coverage increased (+0.009%) to 88.317% when pulling 2e94fad on Arnaud-D:5780-revival into 0eb7143 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.

Deux petites suggestions après une première lecture du code :)

zds/utils/templatetags/humanize_duration.py Outdated Show resolved Hide resolved
templates/tutorialv2/includes/tags_authors.part.html Outdated Show resolved Hide resolved
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.

Il faut effectuer ce changement sinon le filtre n'est pas appliqué.

templates/tutorialv2/includes/tags_authors.part.html Outdated Show resolved Hide resolved
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

@Situphen Situphen merged commit e78c57a into zestedesavoir:dev Oct 12, 2022
@Arnaud-D Arnaud-D deleted the 5780-revival branch October 12, 2022 13:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-Back Concerne le back-end Django hacktoberfest-accepted Pull request approuvée pour le Hacktoberfest
Projects
None yet
Development

Successfully merging this pull request may close these issues.

L'estimation du temps de lecture est ambigü
3 participants