esup_signature: add new-with-workflow endpoint (#77670) #251

Merged
ecazenave merged 3 commits from wip/77670-esup-workflow into main 2023-05-30 14:35:13 +02:00
Owner
No description provided.
ecazenave force-pushed wip/77670-esup-workflow from 46f1bfe2b3 to 9c41159c87 2023-05-17 18:45:48 +02:00 Compare
ecazenave force-pushed wip/77670-esup-workflow from 9c41159c87 to 8d2d2e8d09 2023-05-24 16:33:40 +02:00 Compare
ecazenave force-pushed wip/77670-esup-workflow from f93db8d485 to 1249d45320 2023-05-25 14:27:12 +02:00 Compare
ecazenave changed title from WIP: esup_signature: add new-with-workflow endpoint (#77670) to esup_signature: add new-with-workflow endpoint (#77670) 2023-05-25 14:58:21 +02:00
pmarillonnet reviewed 2023-05-30 11:40:32 +02:00
pmarillonnet left a comment
Owner

Juste quelques petites remarques lors de la relecture, dis-moi ce que tu en penses.

Juste quelques petites remarques lors de la relecture, dis-moi ce que tu en penses.
@ -65,0 +70,4 @@
'all_sign_to_complete': {
'type': 'string',
'description': 'Every recipient has to sign',
'enum': ['true', 'false'],
Owner

Est-ce que pour plus de clarté on n’a pas intérêt à décider d’une valeur par défaut pour tous ces paramètres booléens nouvellement ajoutés au schéma ?

Est-ce que pour plus de clarté on n’a pas intérêt à décider d’une valeur par défaut pour tous ces paramètres booléens nouvellement ajoutés au schéma ?
Author
Owner

Fait.

Fait.
@ -65,0 +106,4 @@
}
SIGN_REQUEST_WTIH_WORKFLOW_SCHEMA = {
Owner

Typo ici (s/WTIH/WITH), à reporter dans le test aussi.

Typo ici (s/WTIH/WITH), à reporter dans le test aussi.
Author
Owner

Corrigé.

Corrigé.
@ -151,0 +270,4 @@
bool_params = {
'all_sign_to_complete': 'allSignToComplete',
'user_sign_first': 'userSignFirst',
'pending': ('pending', True),
Owner

Du détail, mais pas compris pourquoi la valeur par défaut est en dur ici et ne reprend pas celle définie dans le schéma.

Du détail, mais pas compris pourquoi la valeur par défaut est en dur ici et ne reprend pas celle définie dans le schéma.
Author
Owner

Dans la norme JSON schema : "The default keyword specifies a default value. This value is not used to fill in missing values during the validation process."

Charge donc à l'application de "fournir" les valeurs par défaut.

Dans la norme JSON schema : "The default keyword specifies a default value. This value is not used to fill in missing values during the validation process." Charge donc à l'application de "fournir" les valeurs par défaut.
Owner

Cool, j’ignorais ça, merci pour la clarification.

Cool, j’ignorais ça, merci pour la clarification.
ecazenave force-pushed wip/77670-esup-workflow from 1249d45320 to 0c06086585 2023-05-30 13:39:56 +02:00 Compare
ecazenave requested review from pmarillonnet 2023-05-30 14:17:23 +02:00
pmarillonnet approved these changes 2023-05-30 14:20:30 +02:00
ecazenave merged commit 0c06086585 into main 2023-05-30 14:35:13 +02:00
ecazenave deleted branch wip/77670-esup-workflow 2023-05-30 14:35:13 +02:00
Sign in to join this conversation.
No reviewers
No Label
No Milestone
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/passerelle#251
No description provided.