Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[18.0][MIG] mail_optional_follower_notification #1581

Open
wants to merge 48 commits into
base: 18.0
Choose a base branch
from
Open

[18.0][MIG] mail_optional_follower_notification #1581

wants to merge 48 commits into from

Conversation

glitchov
Copy link
Contributor

Migration from 17.0

adrienpeiffer and others added 30 commits February 10, 2025 10:24
@adrienpeiffer @glitchov
Add mail_optional_follower_notification module (OCA#60)
cac69a9 
* [ADD] Add mail_optional_follower_notifications module

* [CHG][mail_optional_follower_notifications] Readme.

* [IMP][mail_optional_follower_notifications] Code review.

* [CHG] Rename mail_optional_follower_notifications to mail_optional_follower_notification

* [IMP][mail_optional_follower_notification] Review
@lmignon @glitchov
Upgrade version from 8 -> 9
f5ed3ee
@lmignon @glitchov
mail.notification has been removed. The notify is done directly on th…
cb6016f 
…e res.partner and channel
@lmignon @glitchov
Tests: Replace XML_IDs no more available in 9.0
aa4cb79
@ThomasBinsfeld @glitchov
Argument auto_commit in send_mail
42270a3
@adrienpeiffer @glitchov
Migration of mail_optional_follower_notification to 10.0
9b49b8d
@adrienpeiffer @glitchov
[FIX] Bad version
b76bd07
@sbidoul @glitchov
[FIX] pep8
ae1374e
@RobinetDenisAcsone @glitchov
[IMP] Hide the checkbox for the mass mailing in mail_optional_followe…
3fde3d5 
…r_notification and mail_optional_autofollow
@adrienpeiffer @glitchov
[FIX] mail_optional_follower_notification: Check force_partners_to_no…
b64efaa 
…tify instead of notify_followers.

Without this commit, followers are NEVER notified when a message is directly posted on the tracker (without click to open the mail compose message form)
@pedrobaeza @glitchov
[FIX] README.rst syntax
d47bc5e
@marylla @glitchov
Translated using Weblate (German)
066c756 
Currently translated at 100.0% (5 of 5 strings)

Translation: social-10.0/social-10.0-mail_optional_follower_notification
Translate-URL: https://translation.odoo-community.org/projects/social-10-0/social-10-0-mail_optional_follower_notification/de/
@pedrocs-exo @glitchov
Translated using Weblate (Portuguese)
e6a2382 
Currently translated at 100.0% (5 of 5 strings)

Translation: social-10.0/social-10.0-mail_optional_follower_notification
Translate-URL: https://translation.odoo-community.org/projects/social-10-0/social-10-0-mail_optional_follower_notification/pt/
@astirpe @glitchov
[12.0][MIG] mail_optional_follower_notification
9a7ecac
@lmignon @glitchov
[IMP] mail_optional_follower_notification: black, isort
9d5ee36
@lmignon @glitchov
[MIG] mail_optional_follower_notification: Migration to 13.0
37300f5
@lmignon @isabellerichard @glitchov
Update mail_optional_follower_notification/models/mail_thread.py
8d469f0 
Co-authored-by: Isabelle RICHARD <[email protected]>
@glitchov
[IMP] pre-commit run -a
e5e39d7
@glitchov
[13.0][FIX] mail_optional_follower_notification: change added span re…
8ad3fb6 
…f in view
@ioans73 @glitchov
[IMP] mail_optional_follower_notification: black, isort, prettier
5f54650
@ioans73 @glitchov
[MIG] mail_optional_follower_notification: Migration to 14.0
3f4fe9d
@oca-travis @glitchov
[UPD] Update mail_optional_follower_notification.pot
82e876d
@OCA-git-bot @glitchov
[UPD] README.rst
e398d4e
@olgamarcocb @glitchov
Translated using Weblate (Spanish)
421ffa2 
Currently translated at 50.0% (2 of 4 strings)

Translation: social-14.0/social-14.0-mail_optional_follower_notification
Translate-URL: https://translation.odoo-community.org/projects/social-14-0/social-14-0-mail_optional_follower_notification/es/
@robin5210 @glitchov
[MIG] mail_optional_follower_notificatio: Migration to 15.0
d3e6f00
@glitchov
[UPD] Update mail_optional_follower_notification.pot
e3ada28
@OCA-git-bot @glitchov
[UPD] README.rst
f67e604
@OCA-git-bot @glitchov
mail_optional_follower_notification 15.0.1.0.1
f1048b4
@OCA-git-bot @glitchov
[IMP] update dotfiles [ci skip]
f5eb5fb
@yankinmax @glitchov
[MIG] mail_optional_follower_notification: Migration to 16.0
1a13222
ajaniszewska-dev and others added 18 commits February 10, 2025 10:24
@ajaniszewska-dev @glitchov
[FIX] fix value returned by _action_send_mail
c50ac3e
@OCA-git-bot @glitchov
mail_optional_follower_notification 16.0.1.0.1
d1e187a
@mymage @glitchov
Translated using Weblate (Italian)
0a56541 
Currently translated at 25.0% (1 of 4 strings)

Translation: social-16.0/social-16.0-mail_optional_follower_notification
Translate-URL: https://translation.odoo-community.org/projects/social-16-0/social-16-0-mail_optional_follower_notification/it/
@OCA-git-bot @glitchov
[UPD] README.rst
3661f03
@Ivorra78 @glitchov
Translated using Weblate (Spanish)
6b88bbf 
Currently translated at 100.0% (4 of 4 strings)

Translation: social-16.0/social-16.0-mail_optional_follower_notification
Translate-URL: https://translation.odoo-community.org/projects/social-16-0/social-16-0-mail_optional_follower_notification/es/
@glitchov
[UPD] Update mail_optional_follower_notification.pot
10fc132
@weblate @glitchov
Update translation files
760a7f4 
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: social-16.0/social-16.0-mail_optional_follower_notification
Translate-URL: https://translation.odoo-community.org/projects/social-16-0/social-16-0-mail_optional_follower_notification/
@mymage @glitchov
Translated using Weblate (Italian)
5be29bb 
Currently translated at 100.0% (4 of 4 strings)

Translation: social-16.0/social-16.0-mail_optional_follower_notification
Translate-URL: https://translation.odoo-community.org/projects/social-16-0/social-16-0-mail_optional_follower_notification/it/
@adrianojprado @glitchov
Translated using Weblate (Portuguese (Brazil))
33fc965 
Currently translated at 100.0% (4 of 4 strings)

Translation: social-16.0/social-16.0-mail_optional_follower_notification
Translate-URL: https://translation.odoo-community.org/projects/social-16-0/social-16-0-mail_optional_follower_notification/pt_BR/
@glitchov
[IMP] mail_optional_follower_notification: pre-commit execution
9c428be
@glitchov
[MIG] mail_optional_follower_notification: Migration to 17.0
ae1b8fe
@glitchov
[UPD] Update mail_optional_follower_notification.pot
338f426
@OCA-git-bot @glitchov
[BOT] post-merge updates
8a0de34
@weblate @glitchov
Update translation files
3638b03 
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: social-17.0/social-17.0-mail_optional_follower_notification
Translate-URL: https://translation.odoo-community.org/projects/social-17-0/social-17-0-mail_optional_follower_notification/
@mymage @glitchov
Translated using Weblate (Italian)
db4da10 
Currently translated at 100.0% (4 of 4 strings)

Translation: social-17.0/social-17.0-mail_optional_follower_notification
Translate-URL: https://translation.odoo-community.org/projects/social-17-0/social-17-0-mail_optional_follower_notification/it/
@AndrzejGerasimukARCHIMEDES @glitchov
Translated using Weblate (Polish)
17fc4f4 
Currently translated at 100.0% (4 of 4 strings)

Translation: social-17.0/social-17.0-mail_optional_follower_notification
Translate-URL: https://translation.odoo-community.org/projects/social-17-0/social-17-0-mail_optional_follower_notification/pl/
@glitchov
[IMP] mail_optional_follower_notification: pre-commit auto fixes
cf395b4
@glitchov
[MIG] mail_optional_follower_notification: Migration to 18.0
e5086b1
CasVissers-360ERP
CasVissers-360ERP approved these changes Mar 14, 2025
View reviewed changes
Copy link

@CasVissers-360ERP CasVissers-360ERP left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Functional review

@OCA-git-bot
Copy link
Contributor

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

@jdidderen jdidderen mentioned this pull request Mar 28, 2025
Draft
@jdidderen
Copy link

Shouldn't this be moved to the new mail repository ?

@jdidderen jdidderen mentioned this pull request Apr 12, 2025
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sbejaoui sbejaoui sbejaoui approved these changes

@CasVissers-360ERP CasVissers-360ERP CasVissers-360ERP approved these changes

Assignees
No one assigned
Labels
approved ready to merge
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.