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

misc: revamp draft stats, replace second part with completion rate (#89282) #1383

Merged
fpeters merged 2 commits from wip/89282-draft-stats-redux into main 2024-04-09 12:07:04 +02:00
Conversation 0 Commits 2 Files Changed 5 +72 -71
fpeters commented 2024-04-09 11:14:47 +02:00
Owner
Copy Link

ce qui donne :

image

qui a été validé avec Anaïs.

ce qui donne : ![image](/attachments/eb0a96c9-ed3e-47ed-b4e3-d76ff02adb6f) qui a été validé avec Anaïs.
image.png
42 KiB
fpeters added 1 commit 2024-04-09 11:14:48 +02:00
fpeters changed title from WIP: misc: revamp draft stats, replace second part with completion rate (#89282) to misc: revamp draft stats, replace second part with completion rate (#89282) 2024-04-09 11:22:58 +02:00
fpeters added 1 commit 2024-04-09 11:27:03 +02:00
gitea/wcs/pipeline/head This commit looks good Details
caaa2ccd7b translation update
tnoel approved these changes 2024-04-09 11:54:31 +02:00
tnoel left a comment
Owner
Copy Link

Validé avec juste une idée de suppression d'un include devenu un peu inutile ; mais je te laisse le choix, c'est pas très important.

Validé avec juste une idée de suppression d'un include devenu un peu inutile ; mais je te laisse le choix, c'est pas très important.
wcs/templates/wcs/backoffice/formdef-inspect.html
@ -116,3 +119,2 @@
<table class="stats completion-rate">
<tbody>
{% for page_drafts in drafts %}
{% include "wcs/backoffice/includes/inspect-draft-by-page.html" with page_id=page_drafts.0 field=page_drafts.1.field percent=page_drafts.1.to_formdata_percent num=page_drafts.1.total den=total_formdata %}
tnoel commented 2024-04-09 11:53:49 +02:00
Owner
Copy Link

Après cette suppression ce include n'est présent qu'une seule fois (~ 5 lignes plus haut), on pourrait penser à le supprimer en intégrant directement son code ici.

Ou pas.

Après cette suppression ce include n'est présent qu'une seule fois (~ 5 lignes plus haut), on pourrait penser à le supprimer en intégrant directement son code ici. Ou pas.
fpeters commented 2024-04-09 12:06:53 +02:00
Author
Owner
Copy Link

Oui j'ai choisi de pas changer ça, pour limiter le code qui bougeait.

Oui j'ai choisi de pas changer ça, pour limiter le code qui bougeait.
fpeters merged commit caaa2ccd7b into main 2024-04-09 12:07:04 +02:00
fpeters deleted branch wip/89282-draft-stats-redux 2024-04-09 12:07:04 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
tnoel
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#1383
No description provided.
Delete Branch "wip/89282-draft-stats-redux"

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?