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

Notifier w.c.s. après maelis (#87168) #471

Merged
bdauvergne merged 3 commits from wip/87168-toulouse-maelis-ne-notifier-w-c into main 2024-02-23 20:20:03 +01:00
Conversation 0 Commits 3 Files Changed 3 +48 -18
bdauvergne commented 2024-02-20 11:22:02 +01:00
Owner
Copy Link
No description provided.
bdauvergne added 3 commits 2024-02-20 11:22:03 +01:00
920a871393 misc: add count parameter to BaseResource.jobs() (#87168) 
It limits the number of jobs to run; it is useful for tests.
gitea/passerelle/pipeline/head This commit looks good Details
ca1e74129d toulouse_maelis: trigger wcs after paid notification (#87168)
bdauvergne requested review from nroche 2024-02-21 12:54:15 +01:00
nroche reviewed 2024-02-21 13:24:32 +01:00
passerelle/base/models.py
@ -608,3 +608,3 @@
return Job.objects.filter(resource_type=resource_type, resource_pk=self.pk)
def jobs(self):
def jobs(self, count=-1):
nroche commented 2024-02-21 13:24:32 +01:00
Owner
Copy Link

Ce paramètre permet de s'arrêter dans les tests avant le lancement d'un second job (crée par le job testé).

Ce paramètre permet de s'arrêter dans les tests avant le lancement d'un second job (crée par le job testé).
nroche approved these changes 2024-02-21 13:25:59 +01:00
nroche left a comment
Owner
Copy Link

Wonderbar, merci Benj !

Le patch notifie Maélis avant de notifier l'usager (wcs).
Il continue d'envoyer le trigger à wcs même si la notif Maélis est en erreur.
Dans ce cas, le trigger n'indiquera pas d'erreur (err=0, pas d'err_desc) mais précisera à wcs l'état de la notification maélis en lui passant aussi les champs concernés.
Il ne change pas le fonctionnement actuel : si l'usager à payé, on continue à lui dire qu'il est bien inscrit.

Wonderbar, merci Benj ! Le patch notifie Maélis avant de notifier l'usager (wcs). Il continue d'envoyer le trigger à wcs même si la notif Maélis est en erreur. Dans ce cas, le trigger n'indiquera pas d'erreur (err=0, pas d'err_desc) mais précisera à wcs l'état de la notification maélis en lui passant aussi les champs concernés. Il ne change pas le fonctionnement actuel : si l'usager à payé, on continue à lui dire qu'il est bien inscrit.
bdauvergne merged commit 668ddf08e5 into main 2024-02-23 20:20:03 +01:00
bdauvergne deleted branch wip/87168-toulouse-maelis-ne-notifier-w-c 2024-02-23 20:20:03 +01:00
Sign in to join this conversation.
Reviewers
No reviewers
nroche
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#471
No description provided.
Delete Branch "wip/87168-toulouse-maelis-ne-notifier-w-c"

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?