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

backoffice: fix card creation in a popup & worflow trace (#74225) #92

Merged
fpeters merged 1 commits from wip/74225-fix-card-creation-in-popup-wf into main 2023-02-06 19:42:55 +01:00
Conversation 1 Commits 1 Files Changed 3 +12 -5
lguerin commented 2023-02-06 18:02:44 +01:00
Owner
Copy Link
No description provided.
lguerin added 1 commit 2023-02-06 18:02:45 +01:00
fpeters approved these changes 2023-02-06 19:39:14 +01:00
fpeters left a comment
Owner
Copy Link

Il m'a fallu un certain temps pour comprendre le diff, qui a l'air tout innocent, donc je me note ici : dans backoffice/data_management.py on redéfinit redirect_after_submitted mais c'était dans cette méthode de FormFillPage que se faisait l'exécution initiale du workflow et la version redéfinie n'incluait pas ça. Plutôt que copier/coller les lignes qui le feraient, simplement ces deux lignes sont exécutées avant de passer la main à l'un ou l'autre redirect_after_submitted.

Pour moi c'est une correction qui peut être passée ce cycle.

Il m'a fallu un certain temps pour comprendre le diff, qui a l'air tout innocent, donc je me note ici : dans backoffice/data_management.py on redéfinit redirect_after_submitted mais c'était dans cette méthode de FormFillPage que se faisait l'exécution initiale du workflow et la version redéfinie n'incluait pas ça. Plutôt que copier/coller les lignes qui le feraient, simplement ces deux lignes sont exécutées avant de passer la main à l'un ou l'autre redirect_after_submitted. Pour moi c'est une correction qui peut être passée ce cycle.
fpeters merged commit b72d603053 into main 2023-02-06 19:42:55 +01:00
fpeters deleted branch wip/74225-fix-card-creation-in-popup-wf 2023-02-06 19:42:55 +01:00
fpeters commented 2023-02-06 19:43:21 +01:00
Owner
Copy Link

Pour moi c'est une correction qui peut être passée ce cycle.

Je l'envoie maintenant comme ça je peux attendre/tagguer dès ce soir.

> Pour moi c'est une correction qui peut être passée ce cycle. Je l'envoie maintenant comme ça je peux attendre/tagguer dès ce soir.
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#92
No description provided.
Delete Branch "wip/74225-fix-card-creation-in-popup-wf"

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?