From c811a248040291a752c7aba558d8c2014cf6c517 Mon Sep 17 00:00:00 2001 From: Pierre-Yves Chibon Date: Dec 10 2018 09:44:44 +0000 Subject: Fix sending notification upon rebase This basically redoes the work from 1ed1ef897a090b5e4513cf7cf53526a39eb1d068 which was undone (by accident) in 4517cde766c746c4897996d5b6ff6ae512c4ded3 I guess because of a wrong rebase or so. Fixes https://pagure.io/pagure/issue/3684 Signed-off-by: Pierre-Yves Chibon --- diff --git a/pagure/lib/notify.py b/pagure/lib/notify.py index bb51610..e782483 100644 --- a/pagure/lib/notify.py +++ b/pagure/lib/notify.py @@ -106,7 +106,7 @@ def blinker_publish(topic, message): ready.send("pagure", topic=topic, message=message) -def log(project, topic, msg, redis=None): +def log(project, topic, msg, webhook=True): """ This is the place where we send notifications to user about actions occuring in pagure. """ @@ -128,7 +128,7 @@ def log(project, topic, msg, redis=None): # Send blink notification to any 3rd party plugins, if there are any blinker_publish(topic, msg) - if redis and project and not project.private: + if webhook and project and not project.private: pagure.lib.tasks_services.webhook_notification.delay( topic=topic, msg=msg,