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

toulouse-maelis: use libelle2 to display activities (#77291) #244

Merged
nroche merged 1 commits from wip/77291-parsifal-using-libelle2 into main 2023-05-12 18:53:32 +02:00
Conversation 0 Commits 1 Files Changed 7 +41 -28
nroche commented 2023-05-09 15:35:25 +02:00
Owner
Copy Link

De façon globale, pour nommer les activités, utiliser le champ libelle2 qui sera mis à disposition partout.
Plus particulièrement :

  • sur l'agenda peri-sco, afficher l'activité et non pas son type,
  • sur l'agenda extra-sco, pour plus de cohérence, avoir le même affichage que ci-dessus même si l'activité est ici implicite.

Patch fait par anticipation que je propose à la relecture, parce que je doute d'obtenir bientôt des données de test.

De façon globale, pour nommer les activités, utiliser le champ libelle2 qui sera mis à disposition partout. Plus particulièrement : * sur l'agenda peri-sco, afficher l'activité et non pas son type, * sur l'agenda extra-sco, pour plus de cohérence, avoir le même affichage que ci-dessus même si l'activité est ici implicite. Patch fait par anticipation que je propose à la relecture, parce que je doute d'obtenir bientôt des données de test.
smihai requested changes 2023-05-09 16:06:52 +02:00
passerelle/contrib/toulouse_maelis/models.py Outdated
@ -1466,3 +1466,3 @@
continue
item['id'] = item['idActivity']
item['text'] = item['libelle']
item['text'] = item.get('libelle2') or item['libelle']
smihai commented 2023-05-09 15:39:30 +02:00
Owner
Copy Link

Ici tu fais get('libelle2') alors que partour ailleurs t'y accèdes avec ['libelle2'].
Il y a ici des situations où la clé ne sera pas retournée dans la réponse ?

Ici tu fais `get('libelle2')` alors que partour ailleurs t'y accèdes avec `['libelle2']`. Il y a ici des situations où la clé ne sera pas retournée dans la réponse ?
nroche commented 2023-05-10 08:46:56 +02:00
Author
Owner
Copy Link

J'anticipe l'amélioration demandée sur readFamily :
à ce jour le libelle2 n'est pas encore retournée dans la réponse du WS readFamily.
https://redmine.sigec.fr/issues/2372

J'anticipe l'amélioration demandée sur readFamily : à ce jour le libelle2 n'est pas encore retournée dans la réponse du WS readFamily. https://redmine.sigec.fr/issues/2372
smihai commented 2023-05-10 10:40:12 +02:00
Owner
Copy Link

Oké

Oké
smihai approved these changes 2023-05-10 10:40:17 +02:00
nroche force-pushed wip/77291-parsifal-using-libelle2 from 8716fbe918 to 135cdbf46a 2023-05-12 18:06:18 +02:00 Compare
nroche merged commit 135cdbf46a into main 2023-05-12 18:53:32 +02:00
nroche deleted branch wip/77291-parsifal-using-libelle2 2023-05-12 18:53:32 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
smihai
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#244
No description provided.
Delete Branch "wip/77291-parsifal-using-libelle2"

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?