entrouvert
/
passerelle
15
0
Fork 0
Code Issues Pull Requests 15 Releases Activity

atal_rest: accept empty files (#81518) #351

Merged
ecazenave merged 1 commits from wip/81518-atal-rest-empty-photos into main 2023-09-25 16:00:35 +02:00
Conversation 1 Commits 1 Files Changed 2 +102 -33
ecazenave commented 2023-09-22 17:03:00 +02:00
Owner
Copy Link
No description provided.
ecazenave force-pushed wip/81518-atal-rest-empty-photos from 1116d56e23 to b700bf900f 2023-09-22 17:24:00 +02:00 Compare
ecazenave changed title from WIP: atal_rest: accept empty files (#81518) to atal_rest: accept empty files (#81518) 2023-09-22 17:33:26 +02:00
ecazenave commented 2023-09-22 17:33:42 +02:00
Author
Owner
Copy Link

En fait deux endpoints impactés.

En fait deux endpoints impactés.
tnoel requested changes 2023-09-25 12:04:57 +02:00
passerelle/apps/atal_rest/models.py Outdated
@ -55,1 +59,3 @@
},
'oneOf': [
FILE_OBJECT,
{'type': 'string', 'description': 'empty file, do not consider'},
tnoel commented 2023-09-25 12:03:52 +02:00
Owner
Copy Link

pour avoir une 400 automatique si quelqu'un tente d'envoyer autre chose qu'une chaine vide, ajouter : « 'pattern': r'^$' » ?

pour avoir une 400 automatique si quelqu'un tente d'envoyer autre chose qu'une chaine vide, ajouter : « 'pattern': r'^$' » ?
passerelle/apps/atal_rest/models.py Outdated
@ -84,1 +78,3 @@
'required': ['content'],
'oneOf': [
FILE_OBJECT,
{'type': 'string', 'description': 'empty file, do not consider'},
tnoel commented 2023-09-25 12:04:23 +02:00
Owner
Copy Link

Même remarque : pour avoir une 400 automatique si quelqu'un tente d'envoyer autre chose qu'une chaine vide, ajouter : « 'pattern': r'^$' » ?

Même remarque : pour avoir une 400 automatique si quelqu'un tente d'envoyer autre chose qu'une chaine vide, ajouter : « 'pattern': r'^$' » ?
ecazenave commented 2023-09-25 12:33:20 +02:00
Author
Owner
Copy Link

Mais du coup tu veux que je vire le {'type': 'null', 'description': 'empty file, do not consider'}, ?

Parce que c'est do copier/coller de ton connecteur !

Mais du coup tu veux que je vire le `{'type': 'null', 'description': 'empty file, do not consider'},` ? Parce que c'est do copier/coller de ton connecteur !
ecazenave commented 2023-09-25 15:56:40 +02:00
Author
Owner
Copy Link

Je raconte n'importe quoi, correction faite.

Je raconte n'importe quoi, correction faite.
ecazenave force-pushed wip/81518-atal-rest-empty-photos from b700bf900f to 424906a3e3 2023-09-25 14:24:41 +02:00 Compare
ecazenave force-pushed wip/81518-atal-rest-empty-photos from 424906a3e3 to 898a14f821 2023-09-25 14:38:38 +02:00 Compare
tnoel approved these changes 2023-09-25 16:00:14 +02:00
ecazenave merged commit 898a14f821 into main 2023-09-25 16:00:35 +02:00
ecazenave deleted branch wip/81518-atal-rest-empty-photos 2023-09-25 16:00:35 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
tnoel
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/passerelle#351
No description provided.
Delete Branch "wip/81518-atal-rest-empty-photos"

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?