openwisp / openwisp-notifications

Notifications module of OpenWISP
https://openwisp.io/docs/dev/notifications/
GNU General Public License v3.0
40 stars 41 forks source link

[feature] Centralize Background API Task Notification Logic #265

Open Aryamanz29 opened 1 year ago

Aryamanz29 commented 1 year ago

We recently introduced new notification types for background API tasks of the ZeroTier VPN backend in openwisp-controller. As part of this implementation, we also created a Celery task class (OpenwispApiTask) that contains the notification logic, along with a retry mechanism. To enhance reusability, we propose centralizing this logic and the new notification types so that other OpenWISP modules can utilize them for background API call Celery tasks.

More Information:

nemesifier commented 2 months ago

We could replace the zerotier notification with the new generic message. What do you think @pandafy?