entrouvert
/
passerelle
16
0
Fork 0
Code Issues Pull Requests 13 Releases Activity

Labels Milestones

toulouse-maelis: pouvoir listers les natures des activités auxquelles un enfant ou un RL est inscrit, par année scolaire (#75549) #161

Merged
nroche merged 1 commits from wip/75549-parsifal-add-year-nature-subscriptions into main 2 months ago
Conversation 1 Commits 1 Files Changed 6
nroche commented 2 months ago
Owner

Ajouter un dictionnaire sur les données des enfants et des responsables légaux, qui indique les codes natures des activités souscrite pour une année scolaire donnée.

Ajouter un dictionnaire sur les données des enfants et des responsables légaux, qui indique les codes natures des activités souscrite pour une année scolaire donnée.
nroche added 9 commits 2 months ago
nroche force-pushed wip/75549-parsifal-add-year-nature-subscriptions from 8dc6021532 to 036d26d4a3 2 months ago
nroche commented 2 months ago
Poster
Owner

(basé sur #75472)

(basé sur #75472)
lguerin reviewed 2 months ago
passerelle/contrib/toulouse_maelis/models.py
@ -305,0 +346,4 @@
continue
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 2 months ago
Owner

ça serait pas plutôt unit.get('dateEnd') ?

ça serait pas plutôt `unit.get('dateEnd')` ?
nroche commented 2 months ago
Poster
Owner

Si, merci l'avoir vu !
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. Dis-moi si tu préfères que je complète les tests.

Si, merci l'avoir vu ! 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. Dis-moi si tu préfères que je complète les tests.
lguerin commented 2 months ago
Owner

Ca serait mieux avec un test plus complet, quand même :)

Ca serait mieux avec un test plus complet, quand même :)
nroche commented 2 months ago
Poster
Owner

Fait (j'ai configuré la restauration scolaire sur 2 année).

Fait (j'ai configuré la restauration scolaire sur 2 année).
lguerin marked this conversation as resolved
nroche force-pushed wip/75549-parsifal-add-year-nature-subscriptions from 036d26d4a3 to 01b03ebf47 2 months ago
nroche force-pushed wip/75549-parsifal-add-year-nature-subscriptions from 01b03ebf47 to da2784ea7e 2 months ago
nroche added 1 commit 2 months ago
lguerin approved these changes 2 months ago
nroche merged commit 50c725d393 into main 2 months ago

Reviewers

lguerin approved these changes 2 months ago
gitea/passerelle/pipeline/head This commit looks good
Details
The pull request has been merged as 50c725d393.
Sign in to join this conversation.
Reviewers
Request review
No reviewers
lguerin
Labels
Apply labels
Clear labels
No items
No Label
Milestone
Set milestone
Clear milestone
No items
No Milestone
Assignees
Assign users
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#161
Write Preview
Loading…
Cancel
Save
There is no content yet.
Delete Branch 'wip/75549-parsifal-add-year-nature-subscriptions'

Deleting a branch is permanent. It CANNOT be undone. Continue?

No
Yes