feat(cof_clubs): Plus de gestion des respoclub #902

Open
lbailly wants to merge 3 commits from rappel_adh into master
Member
No description provided.
lbailly changed title from WIP: feat(cof_clubs): Plus de gestion des respoclub to feat(cof_clubs): Plus de gestion des respoclub 2025-11-24 16:35:54 +01:00
aandres approved these changes 2025-11-24 19:21:57 +01:00
aandres left a comment
Member

La sémantique de "approve" + commentaire est pas claire pour moi mais je veux dire qu'une fois les typos réglées c'est bon pour moi. Je serai curieuse de où se configure le cron (ou assimilé). Tu m'as p-ê déjà montré mais vive les exemples

La sémantique de "approve" + commentaire est pas claire pour moi mais je veux dire qu'une fois les typos réglées c'est bon pour moi. Je serai curieuse de où se configure le cron (ou assimilé). Tu m'as p-ê déjà montré mais vive les exemples
@ -0,0 +15,4 @@
class Command(BaseCommand):
help = "Envoie les mails de rappel au respo de clubs qu'ils doivent adhérer."
Member

aux respos de clubs qu'ils doivent adhérer."

au***x*** respo***s*** de clubs qu'ils doivent adhérer."
lbailly marked this conversation as resolved
@ -0,0 +21,4 @@
def handle(self, *args, **options):
respo_non_cof = User.objects.filter(
profile__is_cof=False,
respo_set__in=Club.objects.filter(status=Club.ClubStatus.ACTIVE),
Member

Je trouve le nom respo_set contre-intuitif mais c'était déjà comme ça donc rip pour moi

Je trouve le nom `respo_set` contre-intuitif mais c'était déjà comme ça donc rip pour moi
@ -0,0 +1,8 @@
Bonjour {{ member.first_name }},
On a remarqué que tu n'était pas membre COF alors que est respo de {% for club in member.respo_set.all %}{% if not forloop.first %}{% if forloop.last %} et {% else %}, {% endif %}{% endif %}{{ club.name }}{% endfor %}.
Member

tu n'étais ... alors que tu es

tu n'étai***s*** ... alors que ***tu es***
lbailly marked this conversation as resolved
@ -0,0 +2,4 @@
On a remarqué que tu n'était pas membre COF alors que est respo de {% for club in member.respo_set.all %}{% if not forloop.first %}{% if forloop.last %} et {% else %}, {% endif %}{% endif %}{{ club.name }}{% endfor %}.
Tu devrais reprendre ton adhésion pour continuer tes activités de respo...
Si c'est une erreur de notre part, nous t'invitons à vite nous envoyer un mails pour qu'on puisse corriger nos petits tableaux.
Member

nous envoyer un mail (sans s)

nous envoyer un mail (sans s)
lbailly marked this conversation as resolved
shared/views.py Outdated
@ -42,3 +42,3 @@
"""
users = User.objects.filter(**self.filters)
users = User.objects.filter(**self.filters).distinct()
Member

Ça c'est un cavalier programmatique on est d'accord :p (en vrai c'est ok juste j'ai mis du temps à capter que c'était effectivement plutôt périphérique)

Ça c'est un cavalier programmatique on est d'accord :p (en vrai c'est ok juste j'ai mis du temps à capter que c'était effectivement plutôt périphérique)
Author
Member

Ça devrait bien se comprendre en suivant les commits (j'ai bien fait un commit distinct pour ça), mais ouai ça m'a fait bizarre d'avoir besoin

Ça devrait bien se comprendre en suivant les commits (j'ai bien fait un commit distinct pour ça), mais ouai ça m'a fait bizarre d'avoir besoin
Author
Member

@aandres wrote in #902 (comment):

La sémantique de "approve" + commentaire est pas claire pour moi mais je veux dire qu'une fois les typos réglées c'est bon pour moi. Je serai curieuse de où se configure le cron (ou assimilé). Tu m'as p-ê déjà montré mais vive les exemples

Pour moi "approve" c'est que t'es ok avec le merge en l'état, donc j'aurais plutôt fait un "request changes", mais l'essentiel c'est que se soit clair, et avec les comment de typos bon...
Pour la config du timer systemd (en tout cas pour l'install DGNum actuelle), je te linkerai la PR infra lié dès que je la fait, et je pense qu'on te fera apply :)

@aandres wrote in https://git.dgnum.eu/DGNum/gestioCOF/pulls/902#issuecomment-19419: > La sémantique de "approve" + commentaire est pas claire pour moi mais je veux dire qu'une fois les typos réglées c'est bon pour moi. Je serai curieuse de où se configure le cron (ou assimilé). Tu m'as p-ê déjà montré mais vive les exemples Pour moi "approve" c'est que t'es ok avec le merge en l'état, donc j'aurais plutôt fait un "request changes", mais l'essentiel c'est que se soit clair, et avec les comment de typos bon... Pour la config du timer systemd (en tout cas pour l'install DGNum actuelle), je te linkerai la PR infra lié dès que je la fait, et je pense qu'on te fera apply :)
Author
Member

DGNum/infrastructure#685
Le premier commit correspond à la prod, le deuxième au staging (sensiblement pareil comme tu t'en doute)
On utilise les timer de systemd, donc des services oneshot lancé selon ce qui est spécifié, la syntaxe de startAt est documenté ici : https://www.freedesktop.org/software/systemd/man/latest/systemd.time.html#Calendar%20Events
Ici j'ai mis juste tout les lundi à heure fixe (donc 1/semaine comme proposé par le cof), pour du 1 toute les 2 semaines je pense que j'aurai fait les 1er et 3e lundi du moi (aka Mon *-*-1..7,15..22 09:31:57), ça colle à peu près
Hésite pas à faire des commentaires sur la PR infra si tu as un doute ofc ou une question, tu pourra la deploy toi même, mais pas merge car reservé au root* de la DGNum

https://git.dgnum.eu/DGNum/infrastructure/pulls/685 Le premier commit correspond à la prod, le deuxième au staging (sensiblement pareil comme tu t'en doute) On utilise les timer de systemd, donc des services oneshot lancé selon ce qui est spécifié, la syntaxe de startAt est documenté ici : https://www.freedesktop.org/software/systemd/man/latest/systemd.time.html#Calendar%20Events Ici j'ai mis juste tout les lundi à heure fixe (donc 1/semaine comme proposé par le cof), pour du 1 toute les 2 semaines je pense que j'aurai fait les 1er et 3e lundi du moi (aka `Mon *-*-1..7,15..22 09:31:57`), ça colle à peu près Hésite pas à faire des commentaires sur la PR infra si tu as un doute ofc ou une question, tu pourra la deploy toi même, mais pas merge car reservé au root* de la DGNum
This pull request can be merged automatically.
You are not authorized to merge this pull request.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin rappel_adh:rappel_adh
git switch rappel_adh
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
3 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
DGNum/gestioCOF!902
No description provided.