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

misc: consider author visibility in hiding evolution entries (#84487) #981

Merged
fpeters merged 1 commits from wip/84487-evolution-ignore-authors-if-not-included into main 2024-01-12 14:43:29 +01:00
Conversation 0 Commits 1 Files Changed 2 +126 -3
fpeters commented 2023-12-31 12:12:23 +01:00
Owner
Copy Link

Honnêtement s'il y a des échanges internes entre agents ils doivent se dérouler dans des statuts privés, les changements d'historique sont visibles aux usagers. Dans le ticket d'origine (pas le ticket technique), la situation particulière est que l'option pour ne pas afficher le nom des intervenants en front est activée, ça laisse ainsi des lignes qui ont juste un timestamp et zéro info supplémentaire.

Ce patch considère cette situation, pour ne pas afficher d'entrées d'historique dans ce cas précis. Dans ce cas précis uniquement.

Honnêtement s'il y a des échanges internes entre agents ils doivent se dérouler dans des statuts privés, les changements d'historique sont visibles aux usagers. Dans le ticket d'origine (pas le ticket technique), la situation particulière est que l'option pour ne pas afficher le nom des intervenants en front est activée, ça laisse ainsi des lignes qui ont juste un timestamp et zéro info supplémentaire. Ce patch considère cette situation, pour ne pas afficher d'entrées d'historique dans ce cas précis. Dans ce cas précis uniquement.
fpeters changed title from WIP: misc: consider author visibility in hiding evolution entries (#84487) to misc: consider author visibility in hiding evolution entries (#84487) 2024-01-05 18:58:19 +01:00
pmarillonnet approved these changes 2024-01-08 09:56:47 +01:00
pmarillonnet left a comment
Owner
Copy Link

Ok, juste une mini-remarque sur le commentaire qui suit la modification. À prendre ou pas. Nonobstant ack :)

Ok, juste une mini-remarque sur le commentaire qui suit la modification. À prendre ou pas. Nonobstant ack :)
wcs/formdata.py
@ -779,3 +785,3 @@
evolution_part.who is None or last_seen_author == evolution_part.who
(evolution_part.who is None or last_seen_author == evolution_part.who) or not include_authors
):
# don't include empty evolution parts if status and author
pmarillonnet commented 2024-01-08 09:55:56 +01:00
Owner
Copy Link

J’en aurais profité pour modifier ce commentaire qui n’est plus entièrement vrai maintenant.

J’en aurais profité pour modifier ce commentaire qui n’est plus entièrement vrai maintenant.
fpeters commented 2024-01-08 10:04:25 +01:00
Author
Owner
Copy Link

Commentaire ajusté.

Commentaire ajusté.
fpeters force-pushed wip/84487-evolution-ignore-authors-if-not-included from b42eab8727 to c370ce4a08 2024-01-08 10:03:50 +01:00 Compare
fpeters merged commit 0eaff91e89 into main 2024-01-12 14:43:29 +01:00
fpeters deleted branch wip/84487-evolution-ignore-authors-if-not-included 2024-01-12 14:43:29 +01:00
Sign in to join this conversation.
Reviewers
No reviewers
pmarillonnet
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#981
No description provided.
Delete Branch "wip/84487-evolution-ignore-authors-if-not-included"

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?