entrouvert/wcs
entrouvert
/
wcs
14
1
Fork 0
Code Issues Pull Requests 34 Releases Activity

ZeroDivisionError sur les stats d'abandon de brouillons (#89270) #1382

Merged
fpeters merged 4 commits from wip/89270-drafts-zero-division-error into main 2024-04-09 10:05:41 +02:00
Conversation 1 Commits 4 Files Changed 5 +49 -19
fpeters commented 2024-04-08 18:41:37 +02:00
Owner
Copy Link

C'était parce que pas de chance j'avais des brouillons qui étaient au-delà de leur durée de vie, et le critère sur la date/heure de réception était appliqué sur l'ensemble des demandes mais pas sur les brouillons.

Faisant ça j'ai aussi ajusté un peu le style en me rendant compte que le pourcentage à 0 (ou bas, de manière générale), n'était pas très lisible (j'ai ajouté un fond léger gris, pour visualiser la zone totale).

Et j'avais fait un commentaire dans la PR d'origine sur la duplication des variables, percent et percent_str, c'était resté et j'ai donc un commit pour réduire ça (en découvrant en lisant la doc qu'il suffit de taper "u" à floatformat pour avoir la version non localisée).

C'était parce que pas de chance j'avais des brouillons qui étaient au-delà de leur durée de vie, et le critère sur la date/heure de réception était appliqué sur l'ensemble des demandes mais pas sur les brouillons. Faisant ça j'ai aussi ajusté un peu le style en me rendant compte que le pourcentage à 0 (ou bas, de manière générale), n'était pas très lisible (j'ai ajouté un fond léger gris, pour visualiser la zone totale). Et j'avais fait un commentaire dans la PR d'origine sur la duplication des variables, percent et percent_str, c'était resté et j'ai donc un commit pour réduire ça (en découvrant en lisant la doc qu'il suffit de taper "u" à floatformat pour avoir la version non localisée).
fpeters added 3 commits 2024-04-08 18:41:37 +02:00
fpeters added 1 commit 2024-04-08 18:46:44 +02:00
fpeters commented 2024-04-08 18:47:19 +02:00
Author
Owner
Copy Link
  • un commit parce qu'il n'y avait pas de test sur la page "Taux sur l’ensemble des demandes de la période".
+ un commit parce qu'il n'y avait pas de test sur la page "Taux sur l’ensemble des demandes de la période".
fpeters changed title from WIP: ZeroDivisionError sur les stats d'abandon de brouillons (#89270) to ZeroDivisionError sur les stats d'abandon de brouillons (#89270) 2024-04-08 18:50:33 +02:00
lguerin approved these changes 2024-04-09 08:49:13 +02:00
fpeters merged commit c8e926afb2 into main 2024-04-09 10:05:41 +02:00
fpeters deleted branch wip/89270-drafts-zero-division-error 2024-04-09 10:05:41 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
lguerin
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: entrouvert/wcs#1382
No description provided.
Delete Branch "wip/89270-drafts-zero-division-error"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?