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

toulouse-maelis: allow activity booking on civil year (#78579) #284

Merged
nroche merged 1 commits from wip/78579-parsifal-extrasco-extend-range-booking into main 2023-06-29 14:11:11 +02:00
Conversation 2 Commits 1 Files Changed 2 +30 -14
nroche commented 2023-06-22 09:34:26 +02:00
Owner
Copy Link

Sur les réservations extra-scolaires (ADL) et loisirs,
faire sauter la contrainte qui impose que les réservations commencent et finissent sur la même année scolaire.

Sur les réservations extra-scolaires (ADL) et loisirs, faire sauter la contrainte qui impose que les réservations commencent et finissent sur la même année scolaire.
nroche commented 2023-06-22 09:39:43 +02:00
Author
Owner
Copy Link

Le patch n'impacte pas les données utilisées par le widget de réservation.

Le patch n'impacte pas les données utilisées par le widget de réservation.
Screenshot 2023-06-22 at 09-10-31 Exemple Inscription ADL (avec agenda) - n°138-3 Titre du site (option de l'inté graphique).png
66 KiB
Screenshot 2023-06-22 at 09-10-31 Exemple Inscription ADL (avec agenda) - n°138-3 Titre du site (option de l'inté graphique).png
nroche commented 2023-06-28 17:29:23 +02:00
Author
Owner
Copy Link

A présent, ça serait utile pour avoir des données de tests qui ne périment pas.
https://redmine.sigec.fr/issues/2681 (voir des données de test tout court)

A présent, ça serait utile pour avoir des données de tests qui ne périment pas. https://redmine.sigec.fr/issues/2681 (voir des données de test tout court)
nroche force-pushed wip/78579-parsifal-extrasco-extend-range-booking from 44037dc593 to 8e41623991 2023-06-29 11:48:24 +02:00 Compare
pmarillonnet approved these changes 2023-06-29 13:52:48 +02:00
pmarillonnet left a comment
Owner
Copy Link

Quelques remarques attrapées au passage.

Quelques remarques attrapées au passage.
tests/test_toulouse_maelis.py Outdated
@ -8884,3 +8906,3 @@
resp = app.get(url + '?family_id=1312', params=params, status=400)
assert resp.json['err'] == 1
assert resp.json['err_desc'] == 'bad date format, should be YYYY-MM-DD'
assert resp.json['err_desc'] == 'invalid value for parameter "start_date (YYYY-MM-DD expected)"'
pmarillonnet commented 2023-06-29 13:52:13 +02:00
Owner
Copy Link

Bizarre ici que l’indice concernant l’erreur de format se retrouve entre les doubles quotes de citation du nom de paramètre. Où la description d’erreur est-elle générée ?

Bizarre ici que l’indice concernant l’erreur de format se retrouve entre les doubles quotes de citation du nom de paramètre. Où la description d’erreur est-elle générée ?
nroche commented 2023-06-29 14:00:10 +02:00
Author
Owner
Copy Link

C'est géré par passerelle, parce que le paramètre est à présent de type "date".
cf passerelle/views.py

C'est géré par passerelle, parce que le paramètre est à présent de type "date". cf passerelle/views.py
tests/test_toulouse_maelis.py Outdated
@ -8890,3 +8912,3 @@
resp = app.get(url + '?family_id=1312', params=params, status=400)
assert resp.json['err'] == 1
assert resp.json['err_desc'] == 'bad date format, should be YYYY-MM-DD'
assert resp.json['err_desc'] == 'invalid value for parameter "end_date (YYYY-MM-DD expected)"'
pmarillonnet commented 2023-06-29 13:52:27 +02:00
Owner
Copy Link

Même remarque ici.

Même remarque ici.
nroche force-pushed wip/78579-parsifal-extrasco-extend-range-booking from 8e41623991 to 0616f216bf 2023-06-29 14:00:57 +02:00 Compare
nroche merged commit 0616f216bf into main 2023-06-29 14:11:11 +02:00
nroche deleted branch wip/78579-parsifal-extrasco-extend-range-booking 2023-06-29 14:11:11 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
pmarillonnet
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#284
No description provided.
Delete Branch "wip/78579-parsifal-extrasco-extend-range-booking"

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?