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

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

Closed
bdauvergne wants to merge 3 commits from wip/87168-toulouse-maelis-ne-notifier-w-c into wip/87030-toulouse-maelis-notifier-les-fac
pull from: wip/87168-toulouse-maelis-ne-notifier-w-c
merge into: entrouvert:wip/87030-toulouse-maelis-notifier-les-fac
Conversation 1 Commits 3 Files Changed 3 +48 -18
bdauvergne commented 2024-02-20 08:00:45 +01:00
Owner
Copy Link
No description provided.
bdauvergne added 4 commits 2024-02-20 08:00:46 +01:00
e8e9523411 misc: add count parameter to BaseResource.jobs() (#87168) 
It limits the number of jobs to run; it is useful for tests.
bdauvergne force-pushed wip/87168-toulouse-maelis-ne-notifier-w-c from d8d1819011 to ca1e74129d 2024-02-20 08:02:48 +01:00 Compare
bdauvergne requested review from nroche 2024-02-20 08:03:09 +01:00
bdauvergne closed this pull request 2024-02-20 11:17:51 +01:00
nroche reviewed 2024-02-21 12:14:51 +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 12:14:51 +01:00
Owner
Copy Link

Ici, ça permet de s'arrêter dans les tests avant le lancement d'un second job (crée par le job testé).

Ici, ça 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 12:19:44 +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.
Le trigger n'indiquera pas d'erreur (err=0, pas d'err_desc) mais précisera l'état de la notification maélis.
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. Le trigger n'indiquera pas d'erreur (err=0, pas d'err_desc) mais précisera l'état de la notification maélis. Il ne change pas le fonctionnement actuel : si l'usager à payé, on continue à lui dire qu'il est bien inscrit.
bdauvergne commented 2024-02-21 13:02:34 +01:00
Author
Owner
Copy Link

Réouverte dans #471

Réouverte dans https://git.entrouvert.org/entrouvert/passerelle/pulls/471
All checks were successful
gitea/passerelle/pipeline/head This commit looks good
Details

Pull request closed

This pull request cannot be reopened because the branch was deleted.
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#470
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?