[18.0][ADD] mail_thread_disable_auto_follower - #206
Conversation
86be9ac to
504d66a
Compare
f1b7d7e to
d7e9952
Compare
|
Is this something different from https://github.com/OCA/mail/tree/18.0/mail_optional_autofollow ? |
@pedrobaeza , Yes, this is a different use case. Here, we are restricting the automatic addition of followers for selected models, whereas mail_optional_autofollow focuses on making the auto-follow behavior optional for users. |
|
OK, then please avoid the usage of plurals in the module name, and clarify in the README/PR the differences for possible reviewers. |
b8acfc7 to
575d725
Compare
575d725 to
0d864d5
Compare
0087f0f to
ab3c237
Compare
@pedrobaeza , Updated the changes, have a look, |
|
It appears a change in nl.po file that shouldn't be there. |
ab3c237 to
ebda739
Compare
@pedrobaeza , Removed the po files, Please have a look. |
pedrobaeza
left a comment
There was a problem hiding this comment.
Formally I have finished my review of the guidelines. About the feature or the code itself, others should do it.
There was a problem hiding this comment.
This icon is the old one. You should remove and let the bots to put the new, or directly include the new.
ebda739 to
d9719be
Compare
This module implements the capability to disable Odoo's automatic follower subscriptions on a per-model basis.