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

atal_rest: start connector (#78904) #291

Merged
ecazenave merged 1 commits from wip/78904-atal-rest into main 2023-06-27 16:45:35 +02:00
Conversation 1 Commits 1 Files Changed 6 +546
ecazenave commented 2023-06-22 18:38:07 +02:00
Owner
Copy Link
No description provided.
ecazenave force-pushed wip/78904-atal-rest from 208698d799 to b86d35ec09 2023-06-22 18:39:07 +02:00 Compare
ecazenave force-pushed wip/78904-atal-rest from b86d35ec09 to 9b4a7e5058 2023-06-22 18:41:07 +02:00 Compare
tnoel reviewed 2023-06-26 11:27:08 +02:00
passerelle/apps/atal_rest/models.py Outdated
@ -0,0 +191,4 @@
'contact/phone': '0101010101',
'contact/zipcode': '75013',
'description': 'some description',
'desired_date': '2023-06-22T13:05:24.360Z',
tnoel commented 2023-06-26 11:26:43 +02:00
Owner
Copy Link

Une date qui est en fait une datetime exprimée en temps universel, ça me semble être un risque (genre des décalages d'une journée entre 22h et minuit)... et c'est pas toujours facile à envoyer depuis wcs. Peut-être qu'il faudrait imposer ici la réception d'une vraie date (YYYY-MM-DD) ?

Une date qui est en fait une datetime exprimée en temps universel, ça me semble être un risque (genre des décalages d'une journée entre 22h et minuit)... et c'est pas toujours facile à envoyer depuis wcs. Peut-être qu'il faudrait imposer ici la réception d'une vraie date (YYYY-MM-DD) ?
ecazenave commented 2023-06-27 14:20:25 +02:00
Author
Owner
Copy Link

Voilà.

Voilà.
ecazenave force-pushed wip/78904-atal-rest from 9b4a7e5058 to 419ab4cd62 2023-06-26 14:50:07 +02:00 Compare
ecazenave force-pushed wip/78904-atal-rest from 419ab4cd62 to e682606ac9 2023-06-26 16:23:06 +02:00 Compare
ecazenave force-pushed wip/78904-atal-rest from e682606ac9 to adaefaad3b 2023-06-26 17:27:51 +02:00 Compare
ecazenave force-pushed wip/78904-atal-rest from adaefaad3b to 44af84fcf8 2023-06-26 18:13:35 +02:00 Compare
ecazenave force-pushed wip/78904-atal-rest from 44af84fcf8 to 403c843aa3 2023-06-27 10:56:19 +02:00 Compare
ecazenave force-pushed wip/78904-atal-rest from 403c843aa3 to 559e354634 2023-06-27 11:23:59 +02:00 Compare
ecazenave force-pushed wip/78904-atal-rest from 559e354634 to 5a4362a3ea 2023-06-27 12:09:36 +02:00 Compare
ecazenave force-pushed wip/78904-atal-rest from 5a4362a3ea to 1effe7abe7 2023-06-27 14:10:56 +02:00 Compare
ecazenave changed title from WIP: atal_rest: start connector (#78904) to atal_rest: start connector (#78904) 2023-06-27 14:20:28 +02:00
ecazenave commented 2023-06-27 14:22:25 +02:00
Author
Owner
Copy Link

Testé en vrai sur l'instance de Saumur, ça me semble ok pour une première version.

(juste pas pu tester jusqu'au bout l'envoi de pièces jointes parce que problème de configuration sur leur instance ATAL mais je pourrai ajuster plus tard si besoin.)

Testé en vrai sur l'instance de Saumur, ça me semble ok pour une première version. (juste pas pu tester jusqu'au bout l'envoi de pièces jointes parce que problème de configuration sur leur instance ATAL mais je pourrai ajuster plus tard si besoin.)
tnoel approved these changes 2023-06-27 14:40:23 +02:00
tnoel left a comment
Owner
Copy Link

J'ai fait un commentaire mais ça passe déjà comme ça imho.

J'ai fait un commentaire mais ça passe déjà comme ça imho.
passerelle/apps/atal_rest/models.py Outdated
@ -0,0 +251,4 @@
except ValueError:
obj = None
if obj is None:
raise APIError("Invalid date: %s" % post_data[param])
tnoel commented 2023-06-27 14:39:26 +02:00
Owner
Copy Link

comme je suis tatasse, tu pourrais faire ici :

raise APIError('%s must be a valid YYYY-MM-DD date (received: "%s")' % (param, post_data[param])

mais je trouve que pour moins te faire chier tu pourrais expliciter l'affaire dans WORKSREQUESTS_SCHEMA (ce qui fait gagner un poil d'auto-documentation dans la page du connecteur)

comme je suis tatasse, tu pourrais faire ici : raise APIError('%s must be a valid YYYY-MM-DD date (received: "%s")' % (param, post_data[param]) mais je trouve que pour moins te faire chier tu pourrais expliciter l'affaire dans WORKSREQUESTS_SCHEMA (ce qui fait gagner un poil d'auto-documentation dans la page du connecteur)
ecazenave commented 2023-06-27 16:45:30 +02:00
Author
Owner
Copy Link

Tenu compte.

Tenu compte.
ecazenave force-pushed wip/78904-atal-rest from 1effe7abe7 to b4f4945ee2 2023-06-27 15:43:37 +02:00 Compare
ecazenave force-pushed wip/78904-atal-rest from b4f4945ee2 to 7212c9056d 2023-06-27 16:15:43 +02:00 Compare
ecazenave merged commit 7212c9056d into main 2023-06-27 16:45:35 +02:00
ecazenave deleted branch wip/78904-atal-rest 2023-06-27 16:45: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#291
No description provided.
Delete Branch "wip/78904-atal-rest"

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?