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

													
				@ -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

Owner

dateEnd ?

nroche commented

Poster

Owner

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

Poster

Owner

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

nroche force-pushed wip/75552-parsifal-subscribed-activity-by-school-year from d2dbbec35f to ee69267875

nroche force-pushed wip/75552-parsifal-subscribed-activity-by-school-year from ee69267875 to f6d6137667

lguerin approved these changes

nroche force-pushed wip/75552-parsifal-subscribed-activity-by-school-year from f6d6137667 to f0be7e0aba

nroche merged commit f0be7e0aba into main 2 months ago

Reviewers

lguerin approved these changes

gitea/passerelle/pipeline/head This commit looks good

Details

The pull request has been merged as f0be7e0aba.

Sign in to join this conversation.

No reviewers

No Label

No Milestone

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.

Loading…

There is no content yet.

Labels Milestones

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

Reviewers