feat(cof_clubs): Plus de gestion des respoclub #902
No reviewers
Labels
No labels
devtype -- backend
devtype -- docs
devtype -- frontend
devtype -- user interface
difficulty -- easy
difficulty -- hard
difficulty -- normal
Doing
domain -- bda
domain -- bds
domain -- cof
domain -- core
domain -- kfet
Good first issue
priority -- high
priority -- low
priority -- medium
priority -- staff-wanted
status -- development
status -- discussion
status -- need review
status -- production
status -- ready to merge
status -- todo
To Do
type -- bug
type -- hygiene
type -- improvement
type -- new feature
bug
duplicate
enhancement
help wanted
invalid
question
wontfix
No milestone
No project
No assignees
3 participants
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
DGNum/gestioCOF!902
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "rappel_adh"
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?
c4d8a7d881tof34c85459cWIP: feat(cof_clubs): Plus de gestion des respoclubto feat(cof_clubs): Plus de gestion des respoclubLa 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."aux respos de clubs qu'ils doivent adhérer."
@ -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),Je trouve le nom
respo_setcontre-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 %}.tu n'étais ... alors que tu es
@ -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.nous envoyer un mail (sans s)
@ -42,3 +42,3 @@"""users = User.objects.filter(**self.filters)users = User.objects.filter(**self.filters).distinct()Ç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 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
f34c85459ctoc6622619ff@aandres wrote in #902 (comment):
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 :)
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èsHé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
View command line instructions
Checkout
From your project repository, check out a new branch and test the changes.