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

misc: skip missing roles when exporting to XML (#24219) #729

Merged
fpeters merged 1 commits from wip/24219-skip-missing-roles-from-workflow-exports into main 2023-10-02 19:11:10 +02:00
Conversation 1 Commits 1 Files Changed 8 +127 -36
fpeters commented 2023-09-27 14:16:08 +02:00
Owner
Copy Link
No description provided.
fpeters added 1 commit 2023-09-27 14:16:08 +02:00
gitea/wcs/pipeline/head Something is wrong with the build of this commit Details
507fba6713 misc: skip missing roles when exporting to XML (#24219)
fpeters force-pushed wip/24219-skip-missing-roles-from-workflow-exports from 507fba6713 to 5ea143f6f4 2023-09-27 16:54:48 +02:00 Compare
fpeters force-pushed wip/24219-skip-missing-roles-from-workflow-exports from 5ea143f6f4 to 04d21ef1dd 2023-09-27 17:16:18 +02:00 Compare
fpeters force-pushed wip/24219-skip-missing-roles-from-workflow-exports from 04d21ef1dd to f48ec527d1 2023-09-27 21:18:51 +02:00 Compare
fpeters force-pushed wip/24219-skip-missing-roles-from-workflow-exports from f48ec527d1 to f2044cb833 2023-09-28 08:09:28 +02:00 Compare
fpeters force-pushed wip/24219-skip-missing-roles-from-workflow-exports from f2044cb833 to 503f2c25a2 2023-09-28 08:37:07 +02:00 Compare
fpeters commented 2023-09-28 09:35:31 +02:00
Author
Owner
Copy Link

Le point problématique ici serait sur l'action d'affichage d'un message, le comportement actuel en l'absence de rôle spécifié est d'afficher le message; avant ce patch un workflow avec une action d'affichage de message limitée à un rôle qui n'existe plus, s'il était dupliqué, on avait une erreur. Désormais ça va passer et le message sera affiché à tout le monde. Je peux imaginer ce cas assez particulier et qu'il est raisonnable de l'ignorer.

Le point problématique ici serait sur l'action d'affichage d'un message, le comportement actuel en l'absence de rôle spécifié est d'afficher le message; avant ce patch un workflow avec une action d'affichage de message limitée à un rôle qui n'existe plus, s'il était dupliqué, on avait une erreur. Désormais ça va passer et le message sera affiché à tout le monde. Je peux imaginer ce cas assez particulier et qu'il est raisonnable de l'ignorer.
fpeters changed title from WIP: misc: skip missing roles when exporting to XML (#24219) to misc: skip missing roles when exporting to XML (#24219) 2023-09-28 09:35:34 +02:00
lguerin approved these changes 2023-10-02 18:45:40 +02:00
tests/admin_pages/test_workflow.py Outdated
@ -3757,0 +3795,4 @@
action.dispatch_type = 'automatic'
workflow.store()
resp = app.get('/backoffice/workflows/%s/inspect' % workflow.id)
assert resp.pyquery('.rules').text() == 'foo → Unknown role (1)'
lguerin commented 2023-10-02 18:43:31 +02:00
Owner
Copy Link

1, c'est l'id du rôle ?

1, c'est l'id du rôle ?
fpeters commented 2023-10-02 18:51:24 +02:00
Author
Owner
Copy Link

Oui, je vais appliquer ce changement avant de merger, pour que ça soit plus clair :

-    assert resp.pyquery('.rules').text() == 'foo → Unknown role (1)'
+    assert resp.pyquery('.rules').text() == f'foo → Unknown role ({role.id})'
Oui, je vais appliquer ce changement avant de merger, pour que ça soit plus clair : ``` - assert resp.pyquery('.rules').text() == 'foo → Unknown role (1)' + assert resp.pyquery('.rules').text() == f'foo → Unknown role ({role.id})' ```
fpeters force-pushed wip/24219-skip-missing-roles-from-workflow-exports from 503f2c25a2 to 15f4e18ac2 2023-10-02 18:51:39 +02:00 Compare
fpeters merged commit e2f166b8bd into main 2023-10-02 19:11:10 +02:00
fpeters deleted branch wip/24219-skip-missing-roles-from-workflow-exports 2023-10-02 19:11:10 +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#729
No description provided.
Delete Branch "wip/24219-skip-missing-roles-from-workflow-exports"

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?