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

backoffice: inspect, link to action for button wf trace (#86575) #1107

Merged
lguerin merged 1 commits from wip/86575-display-action-id-in-trace into main 2024-02-09 11:56:57 +01:00
Conversation 0 Commits 1 Files Changed 6 +71 -9
lguerin commented 2024-02-08 09:52:54 +01:00
Owner
Copy Link

branche basée sur #86597

proposition: un lien pour aller sur l'action en question

branche basée sur #86597 proposition: un lien pour aller sur l'action en question
wf-trace-button.png
33 KiB
wf-trace-button.png
lguerin added 1 commit 2024-02-08 09:52:54 +01:00
lguerin force-pushed wip/86575-display-action-id-in-trace from dbc679ec6d to be7b7be262 2024-02-08 16:03:42 +01:00 Compare
lguerin reviewed 2024-02-08 16:07:14 +01:00
wcs/wf/choice.py Outdated
@ -88,6 +88,9 @@ class ChoiceWorkflowStatusItem(WorkflowStatusJumpItem):
else:
return _('not completed')
def get_line_short_details(self):
lguerin commented 2024-02-08 16:04:35 +01:00
Author
Owner
Copy Link

pour ne pas avoir tout le détail du statut cible, des rôles, etc

pour ne pas avoir tout le détail du statut cible, des rôles, etc
wcs/workflows.py Outdated
@ -1730,6 +1730,9 @@ class WorkflowGlobalActionManualTrigger(WorkflowGlobalActionTrigger):
parts.append(_('not assigned'))
return ', '.join([str(x) for x in parts])
def render_as_short_line(self):
lguerin commented 2024-02-08 16:06:13 +01:00
Author
Owner
Copy Link

J'ai défini des méthodes render_as_short_line là où il y avait des render_as_line, sans trop savoir si ça serait appelé dans l'inspect ou pas. J'ai pensé qu'il était mieux d'avoir des méthodes inutiles, plutôt que trop d'informations dans le suivi des actions.

J'ai défini des méthodes render_as_short_line là où il y avait des render_as_line, sans trop savoir si ça serait appelé dans l'inspect ou pas. J'ai pensé qu'il était mieux d'avoir des méthodes inutiles, plutôt que trop d'informations dans le suivi des actions.
wcs/workflows.py Outdated
@ -3002,1 +3016,4 @@
def render_as_short_line(self):
label = self.description
if hasattr(self, 'get_line_short_details'):
lguerin commented 2024-02-08 16:07:08 +01:00
Author
Owner
Copy Link

Si la méthode get_line_short_details n'est pas définie, on n'affiche rien de plus que la description de l'action. On pourra toujours définir des méthodes à la demande.

Si la méthode get_line_short_details n'est pas définie, on n'affiche rien de plus que la description de l'action. On pourra toujours définir des méthodes à la demande.
lguerin force-pushed wip/86575-display-action-id-in-trace from be7b7be262 to 4205f14efe 2024-02-08 16:18:22 +01:00 Compare
fpeters approved these changes 2024-02-09 08:45:51 +01:00
tests/workflow/test_jump.py Outdated
@ -579,3 +579,3 @@
'Created (backoffice submission)',
'Status1',
'Timeout jump',
'Timeout jump - Change Status on Timeout',
fpeters commented 2024-02-09 08:45:27 +01:00
Owner
Copy Link

Ça fait un peu redondant/inutile mais ne complexifions pas juste pour ça.

Ça fait un peu redondant/inutile mais ne complexifions pas juste pour ça.
lguerin force-pushed wip/86575-display-action-id-in-trace from 4205f14efe to c28b82745d 2024-02-09 10:47:02 +01:00 Compare
lguerin merged commit c28b82745d into main 2024-02-09 11:56:57 +01:00
lguerin deleted branch wip/86575-display-action-id-in-trace 2024-02-09 11:56:57 +01:00
Sign in to join this conversation.
Reviewers
No reviewers
fpeters
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#1107
No description provided.
Delete Branch "wip/86575-display-action-id-in-trace"

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?