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

toulouse-maelis: ajouter un paramètre année scolaire au endpoint qui liste les activités souscrites (#75552) #162

Merged
nroche merged 1 commits from wip/75552-parsifal-subscribed-activity-by-school-year into main 2023-03-30 11:08:55 +02:00
Conversation 1 Commits 1 Files Changed 2 +36 -1
nroche commented 2023-03-26 15:23:25 +02:00
Owner
Copy Link

Pour ne lister que les activités souscrite sur une année scolaire.

Pour ne lister que les activités souscrite sur une année scolaire.
nroche added 9 commits 2023-03-26 15:23:25 +02:00
nroche commented 2023-03-26 15:23:48 +02:00
Author
Owner
Copy Link

(basé sur #75472)

(basé sur #75472)
lguerin reviewed 2023-03-27 19:05:06 +02:00
passerelle/contrib/toulouse_maelis/models.py
@ -1164,0 +1262,4 @@
school_years = set()
for unit in item['subscribesUnit']:
start_year = utils.get_reference_year_from_date(unit.get('dateStart'))
end_year = utils.get_reference_year_from_date(unit.get('dateStart'))
lguerin commented 2023-03-27 19:04:25 +02:00
Owner
Copy Link

dateEnd ?

dateEnd ?
nroche commented 2023-03-29 15:17:59 +02:00
Author
Owner
Copy Link

Si, merci l'avoir vu !

(comme pour #161)
Je n'ai pas complété les tests parce que bien que j'ai déjà rencontré ce cas d'une activité qui court sur plusieurs années scolaires, je pense qu'il s'agit d'une erreur de configuration (je n'ai pas eu de réponse à ce sujet).
J'ai testé à la main pour être sûr de ne rien avoir laissé passé d'autre ici, mais dis-moi si tu préfères que je complète les tests.

Si, merci l'avoir vu ! (comme pour https://git.entrouvert.org/entrouvert/passerelle/pulls/161) Je n'ai pas complété les tests parce que bien que j'ai déjà rencontré ce cas d'une activité qui court sur plusieurs années scolaires, je pense qu'il s'agit d'une erreur de configuration (je n'ai pas eu de réponse à ce sujet). J'ai testé à la main pour être sûr de ne rien avoir laissé passé d'autre ici, mais dis-moi si tu préfères que je complète les tests.
nroche commented 2023-03-29 18:02:29 +02:00
Author
Owner
Copy Link

Test ajouté (en rebasant la PR sur #161).

Test ajouté (en rebasant la PR sur https://git.entrouvert.org/entrouvert/passerelle/pulls/161).
lguerin marked this conversation as resolved
nroche force-pushed wip/75552-parsifal-subscribed-activity-by-school-year from 4f34f25ba1 to d2dbbec35f 2023-03-29 14:50:50 +02:00 Compare
nroche force-pushed wip/75552-parsifal-subscribed-activity-by-school-year from d2dbbec35f to ee69267875 2023-03-29 17:35:58 +02:00 Compare
nroche force-pushed wip/75552-parsifal-subscribed-activity-by-school-year from ee69267875 to f6d6137667 2023-03-29 18:40:46 +02:00 Compare
lguerin approved these changes 2023-03-30 08:45:30 +02:00
nroche force-pushed wip/75552-parsifal-subscribed-activity-by-school-year from f6d6137667 to f0be7e0aba 2023-03-30 10:00:35 +02:00 Compare
nroche merged commit f0be7e0aba into main 2023-03-30 11:08:55 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
lguerin
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#162
No description provided.
Delete Branch "wip/75552-parsifal-subscribed-activity-by-school-year"

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?