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

toulouse-axel: fix possible days endpoint (#78903) #288

Merged
lguerin merged 1 commits from wip/78903-toulouse-axel-fix-possible-days into main 2023-06-30 08:58:50 +02:00
Conversation 0 Commits 1 Files Changed 2 +66 -6
lguerin commented 2023-06-22 16:16:47 +02:00
Owner
Copy Link

La spec dit que JOUR doit contenir 5 caractères, 0 ou 1, le xsd dit:
<xsd:element name="JOUR" type="xsd:string"/>

La spec dit que JOUR doit contenir 5 caractères, 0 ou 1, le xsd dit: `<xsd:element name="JOUR" type="xsd:string"/>`
fpeters reviewed 2023-06-29 18:36:19 +02:00
tests/test_toulouse_axel.py Outdated
@ -3344,0 +3379,4 @@
<ACTIVITE>
<ANNEEREFERENCE>2019</ANNEEREFERENCE>
<IDACTIVITE>A19P1M1</IDACTIVITE>
<JOUR>foo</JOUR>
fpeters commented 2023-06-29 18:36:19 +02:00
Owner
Copy Link

Dans ce qu'ils envoient, (j'ai du passer par une trace par email, dans sentry la valeur de booking n'était pas présente), j'ai

{'ACTIVITE': [{'ANNEEREFERENCE': '2022',
               'IDACTIVITE': 'A22P1M1JOL',
               'JOUR': '2'},
              {'ANNEEREFERENCE': '2022',
               'IDACTIVITE': 'A22P1M2JOL',
               'JOUR': '2'},
              {'ANNEEREFERENCE': '2022',
               'IDACTIVITE': 'A22P1M3JOL',
               'JOUR': '2'},
              {'ANNEEREFERENCE': '2022',
               'IDACTIVITE': 'A22P1M4JOL',
               'JOUR': '2'}],
 'IDPERSONNE': '825717'}

Plutôt que "foo" qui ne correspond à rien et posé ici dans les tests, tu pourrais mettre l'exemple ici, avec un commentaire,

        <JOUR>2</JOUR>  <!-- invalid value returned by axel, five digits are required -->
Dans ce qu'ils envoient, (j'ai du passer par une trace par email, dans sentry la valeur de booking n'était pas présente), j'ai ``` {'ACTIVITE': [{'ANNEEREFERENCE': '2022', 'IDACTIVITE': 'A22P1M1JOL', 'JOUR': '2'}, {'ANNEEREFERENCE': '2022', 'IDACTIVITE': 'A22P1M2JOL', 'JOUR': '2'}, {'ANNEEREFERENCE': '2022', 'IDACTIVITE': 'A22P1M3JOL', 'JOUR': '2'}, {'ANNEEREFERENCE': '2022', 'IDACTIVITE': 'A22P1M4JOL', 'JOUR': '2'}], 'IDPERSONNE': '825717'} ``` Plutôt que "foo" qui ne correspond à rien et posé ici dans les tests, tu pourrais mettre l'exemple ici, avec un commentaire, ``` <JOUR>2</JOUR> <!-- invalid value returned by axel, five digits are required --> ```
lguerin commented 2023-06-29 18:59:54 +02:00
Author
Owner
Copy Link

ok

ok
lguerin marked this conversation as resolved
lguerin force-pushed wip/78903-toulouse-axel-fix-possible-days from b50ab3f971 to f928a10fc5 2023-06-29 18:59:15 +02:00 Compare
fpeters approved these changes 2023-06-29 19:28:43 +02:00
lguerin merged commit f928a10fc5 into main 2023-06-30 08:58:50 +02:00
lguerin deleted branch wip/78903-toulouse-axel-fix-possible-days 2023-06-30 08:58:51 +02:00
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/passerelle#288
No description provided.
Delete Branch "wip/78903-toulouse-axel-fix-possible-days"

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?