[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
|
Hi @rvalyi, @ovnicraft, Can you please review this? |
This module implements the capability to disable Odoo's automatic follower subscriptions on a per-model basis.