You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
For now that this project is monolithic it's ok to have a notification system implemented within the project. However later on this needs to be sent to a celery task.
It's better to have an abstract class for notification system in case it grows into multiple types of notification (SMS, Email, Push notification, etc.)
Notification when order is set (with an order id)
Notification when order is ready
Scheduled notifications for previous customers (based on specific statistics) (using celery)
The text was updated successfully, but these errors were encountered:
This task and the branch notification-system branch will be pended since the notification system is based on User model in the database which currently does not exist.
Once the #181 is completed (merged to develop branch), we will merge develop with notification-system and continue this task.
For now that this project is monolithic it's ok to have a notification system implemented within the project. However later on this needs to be sent to a celery task.
It's better to have an abstract class for notification system in case it grows into multiple types of notification (SMS, Email, Push notification, etc.)
The text was updated successfully, but these errors were encountered: