[17.0][MIG] mail_show_follower: Migration to 17.0 #1570
+1,631
−0
Conversation
This file contains 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
ArnauCForgeFlow
force-pushed
the
17.0-mig-mail_show_follower
branch
2 times, most recently
from
220cbc3 to
feab6dc
Compare
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#788 the PR should be merged manually but we should update the version of the module as a minor update
Co-authored-by: Miquel Raïch <[email protected]>
[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/
Currently translated at 62.5% (10 of 16 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/es/
Currently translated at 100.0% (16 of 16 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/es/
Currently translated at 25.0% (4 of 16 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 37.5% (6 of 16 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% (16 of 16 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% (16 of 16 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/
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/
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/it/
…sages correctly + proper lang handle In a multi-company environment, if one company uses the show_followers_partner_format by partner_name and another company uses partner_email, a user with permissions in both companies may experience issues. If a document belongs to Company A, but the current company is set to Company B, the message format is taken from the current company (Company B) instead of the document’s company (Company A). This fix ensures that the message format is rendered based on the document's company. It also fixes the lang handling for cases with False values.
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/
ArnauCForgeFlow
force-pushed
the
17.0-mig-mail_show_follower
branch
from
feab6dc to
ce7e05e
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
This PR supersedes #1440