Skip to content
Snippets Groups Projects
Commit 3ed68285 authored by Jonathan Weth's avatar Jonathan Weth :keyboard:
Browse files

Merge branch...

Merge branch '835-notifaction-send-always-creates-celery-task-with-force-sending' into 'release-3.0'

Resolve "Notifaction.send always creates Celery task with force sending"

See merge request !1212
parents e24b720f 1399f251
No related branches found
No related tags found
3 merge requests!1237Release 3.0,!1212Resolve "Notifaction.send always creates Celery task with force sending",!1183Release 3.0
Pipeline #124076 canceled
......@@ -35,6 +35,7 @@ Fixed
* Route changes in the Legacy-Component iframe didn't trigger a scroll to the top
* Query strings did not get passed when navigating legacy pages inside of the SPA.
* Retry button on error 500 page did not trigger a reload of the page.
* When the Celery worker wasn't able to execute all tasks in time, notifications were sent multiple times.
`3.0b3`_ - 2023-03-19
---------------------
......
......@@ -769,7 +769,7 @@ class Notification(ExtensibleModel, TimeStampedModel):
def send(self, resend: bool = False) -> Optional[AsyncResult]:
"""Send the notification to the recipient."""
if not self.sent or resend:
return send_notification.delay(self.pk, resend=True)
return send_notification.delay(self.pk, resend=resend)
class Meta:
verbose_name = _("Notification")
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment