-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
[16.0][ADD] base_notification #3425
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: 16.0
Are you sure you want to change the base?
Conversation
e7ed74e to
2eafb6a
Compare
05490c9 to
7ae238b
Compare
|
@sebastienbeau I need your opinion about this new module? |
c7bdd12 to
a39a093
Compare
a39a093 to
662df47
Compare
662df47 to
5bb4e70
Compare
|
Why not add a notification action option to the existing Automated Actions? |
Thanks MR. Daniel, I was asking myself what is the difference now after making it. I will extend the automated action in a new PR to compare and to review. |
|
This is nice! We have built a similar functionality (still in progress) that is actually connecting to new unified notification system (email, SMS...etc). One important point to mention, ensure you have safe net around core ORM calls i.e. extending create/write/unlink using |




if a sales order is under many validation steps sometimes the nitifactions are sent, but if the order is following different routes not perfectly that is coded we will not be notified.
The approach to have a precise notification handling methods based on all probable things ORM like create, update, delete.