esup_signature: add new-with-workflow endpoint (#77670) #251
No reviewers
Labels
No Label
No Milestone
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: entrouvert/passerelle#251
Loading…
Reference in New Issue
No description provided.
Delete Branch "wip/77670-esup-workflow"
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?
46f1bfe2b3
to9c41159c87
9c41159c87
to8d2d2e8d09
f93db8d485
to1249d45320
WIP: esup_signature: add new-with-workflow endpoint (#77670)to esup_signature: add new-with-workflow endpoint (#77670)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'],
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 ?
Fait.
@ -65,0 +106,4 @@
}
SIGN_REQUEST_WTIH_WORKFLOW_SCHEMA = {
Typo ici (s/WTIH/WITH), à reporter dans le test aussi.
Corrigé.
@ -151,0 +270,4 @@
bool_params = {
'all_sign_to_complete': 'allSignToComplete',
'user_sign_first': 'userSignFirst',
'pending': ('pending', True),
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.
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.
Cool, j’ignorais ça, merci pour la clarification.
1249d45320
to0c06086585