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

toulouse-maelis: filter subscribable school years (#83262) #395

Merged
nroche merged 1 commits from wip/83262-parsifal-subscibable-school-years into main 2023-11-10 12:11:29 +01:00
Conversation 0 Commits 1 Files Changed 4 +72 -16
nroche commented 2023-11-09 10:57:46 +01:00
Owner
Copy Link

Ne présenter que les années scolaires ouvertes à la souscription.

Ne présenter que les années scolaires ouvertes à la souscription.
nroche added 1 commit 2023-11-09 10:57:47 +01:00
gitea/passerelle/pipeline/head This commit looks good Details
10ce2295cc toulouse-maelis: filter subscribable school years (#83262)
nroche force-pushed wip/83262-parsifal-subscibable-school-years from 10ce2295cc to c52e61b534 2023-11-10 10:40:16 +01:00 Compare
smihai requested changes 2023-11-10 11:20:35 +01:00
passerelle/contrib/toulouse_maelis/models.py Outdated
@ -2633,2 +2634,4 @@
description='Lister les années scolaires',
name='read-school-years-list',
parameters={
'subscribable': {
smihai commented 2023-11-10 11:15:41 +01:00
Owner
Copy Link

En anglais le terme correct serait "subscriptable" il me semble.

En anglais le terme correct serait "subscriptable" il me semble.
nroche commented 2023-11-10 11:37:20 +01:00
Author
Owner
Copy Link

En comparant ici, j'ai l'impression que l'idée d'inscription / souscription colle mieux à cette seconde orthographe
https://www.linguee.fr/anglais-francais/traduction/subscriptable.html

Essayez avec cette orthographe : « subscribable »

En comparant ici, j'ai l'impression que l'idée d'inscription / souscription colle mieux à cette seconde orthographe https://www.linguee.fr/anglais-francais/traduction/subscriptable.html > Essayez avec cette orthographe : « subscribable »
smihai commented 2023-11-10 11:55:10 +01:00
Owner
Copy Link

Ok, fais comme tu le sens.

Ok, fais comme tu le sens.
passerelle/contrib/toulouse_maelis/models.py Outdated
@ -2638,0 +2649,4 @@
referential = self.get_referential('YearSchool')
data = []
for item in referential:
if bool(subscribable) is True:
smihai commented 2023-11-10 11:17:17 +01:00
Owner
Copy Link

strtobool retournera les entiers 0 ou 1, donc pas besoin de conversion en booléen ici.

`strtobool` retournera les entiers 0 ou 1, donc pas besoin de conversion en booléen ici.
nroche commented 2023-11-10 11:28:35 +01:00
Author
Owner
Copy Link

Ça m'a surpris aussi, mais malgré son nom, strtobool renvoi un entier.
https://github.com/python/cpython/blob/v3.11.2/Lib/distutils/util.py#L308

Cependant, je réalise que cette fonction va être dépréciée :
https://peps.python.org/pep-0632/

For these functions, you will need to reimplement the functionality yourself.

Bref, je simplifie.

Ça m'a surpris aussi, mais malgré son nom, strtobool renvoi un entier. https://github.com/python/cpython/blob/v3.11.2/Lib/distutils/util.py#L308 Cependant, je réalise que cette fonction va être dépréciée : https://peps.python.org/pep-0632/ > For these functions, you will need to reimplement the functionality yourself. Bref, je simplifie.
smihai commented 2023-11-10 11:47:03 +01:00
Owner
Copy Link

Bref, je simplifie.

Tu peux juste attendre "yes" pour vrai et considérer que tout le reste est faux.

> > Bref, je simplifie. Tu peux juste attendre "yes" pour vrai et considérer que tout le reste est faux.
nroche commented 2023-11-10 12:04:09 +01:00
Author
Owner
Copy Link

J'ai finalement recopié le code pour avoir un appel relativement clair.
Parce que ici j'ai l'impression que ça complique la compréhension d'avoir un paramètre qui est par défaut à vrai.

J'ai finalement recopié le code pour avoir un appel relativement clair. Parce que ici j'ai l'impression que ça complique la compréhension d'avoir un paramètre qui est par défaut à vrai.
nroche force-pushed wip/83262-parsifal-subscibable-school-years from c52e61b534 to 3b156308a6 2023-11-10 11:54:51 +01:00 Compare
nroche force-pushed wip/83262-parsifal-subscibable-school-years from 3b156308a6 to 97b04c728d 2023-11-10 11:57:32 +01:00 Compare
smihai approved these changes 2023-11-10 12:05:57 +01:00
nroche merged commit 81f58cad59 into main 2023-11-10 12:11:29 +01:00
nroche deleted branch wip/83262-parsifal-subscibable-school-years 2023-11-10 12:11:29 +01: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#395
No description provided.
Delete Branch "wip/83262-parsifal-subscibable-school-years"

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?