[19.0][MIG] mail_show_follower: Migration to 19.0#197
Open
apik-mgu wants to merge 61 commits into
Open
Conversation
Display CC according to notification_type
Currently translated at 100.0% (7 of 7 strings) Translation: social-12.0/social-12.0-mail_show_follower Translate-URL: https://translation.odoo-community.org/projects/social-12-0/social-12-0-mail_show_follower/fr/
As commented in OCA/social#788 the PR should be merged manually but we should update the version of the module as a minor update
[REF] Compact comment
Added settings to customize notification and translations to important parts of the message. [FIX] Security fixes and simplified customization Field `show_followers_partner_format` formatted with %(param)s Removed CSS customizations [FIX] README options [FIX] Index.html configure [FIX] Several changes - Removed unnecesary properties on res_config_settings - Markup safe on partner_format - Use email_domain_extract instead own one
When using composer, doesn't include all recipients of the message on the notification.
In the highly improbable but actually possible and real world case that a partner has more than one user associated, this change makes the emails able to send. @moduon MT-1634
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: social-16.0/social-16.0-mail_show_follower Translate-URL: https://translation.odoo-community.org/projects/social-16-0/social-16-0-mail_show_follower/
This module aims to show the recipients of an email in the email header, but the previous implementation was not suitable for internal notes. When **logging an internal note** with tagged users and/or partners, those specific recipients should be displayed as well as the document's followers subscribed to this message subtype. When **sending an email**, the recipients should be the document's followers. This is achieved by using the union of `mail.mail`'s `recipient_ids` and `notified_partner_ids`.
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: social-16.0/social-16.0-mail_show_follower Translate-URL: https://translation.odoo-community.org/projects/social-16-0/social-16-0-mail_show_follower/
Currently translated at 100.0% (17 of 17 strings) Translation: social-16.0/social-16.0-mail_show_follower Translate-URL: https://translation.odoo-community.org/projects/social-16-0/social-16-0-mail_show_follower/pt_BR/
Slides and forums are also places where you don't want to show the recipients.
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: social-16.0/social-16.0-mail_show_follower Translate-URL: https://translation.odoo-community.org/projects/social-16-0/social-16-0-mail_show_follower/
Currently translated at 100.0% (21 of 21 strings) Translation: social-16.0/social-16.0-mail_show_follower Translate-URL: https://translation.odoo-community.org/projects/social-16-0/social-16-0-mail_show_follower/it/
Currently translated at 100.0% (21 of 21 strings) Translation: social-16.0/social-16.0-mail_show_follower Translate-URL: https://translation.odoo-community.org/projects/social-16-0/social-16-0-mail_show_follower/pt_BR/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: social-17.0/social-17.0-mail_show_follower Translate-URL: https://translation.odoo-community.org/projects/social-17-0/social-17-0-mail_show_follower/
Before this commit, 'Show Internal Users CC' would always be False, regardless of the setting in the company, in case the message came from a record with a company_id field but without a value assignment to it (e.g., a partner record with no company), as company would be an empty record. This commit fixes the behavior by capturing company from self.env in these cases.
Currently translated at 100.0% (25 of 25 strings) Translation: mail-18.0/mail-18.0-mail_show_follower Translate-URL: https://translation.odoo-community.org/projects/mail-18-0/mail-18-0-mail_show_follower/it/
Currently translated at 28.0% (7 of 25 strings) Translation: mail-18.0/mail-18.0-mail_show_follower Translate-URL: https://translation.odoo-community.org/projects/mail-18-0/mail-18-0-mail_show_follower/fr/
Currently translated at 100.0% (25 of 25 strings) Translation: mail-18.0/mail-18.0-mail_show_follower Translate-URL: https://translation.odoo-community.org/projects/mail-18-0/mail-18-0-mail_show_follower/ca/
Currently translated at 92.0% (23 of 25 strings) Translation: mail-18.0/mail-18.0-mail_show_follower Translate-URL: https://translation.odoo-community.org/projects/mail-18-0/mail-18-0-mail_show_follower/de/
Currently translated at 100.0% (25 of 25 strings) Translation: mail-18.0/mail-18.0-mail_show_follower Translate-URL: https://translation.odoo-community.org/projects/mail-18-0/mail-18-0-mail_show_follower/es/
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
20 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.