Saturday, November 23, 2019
3 changes · master
Miscellaneous changes
Description of the issue/feature this PR addresses: If the user put more than 200 emojis, we didn't replace emoji with image. But due to a typo in the code, this mechanism broke the discuss application. Current behavior before PR: If the user put more than 200 emojis, the discuss app is broken Desired behavior after PR is merged: If the user put more than 200 emojis, it's working. -- I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr Forward-
Original PR description
Description of the issue/feature this PR addresses: If the user put more than 200 emojis, we didn't replace emoji with image. But due to a typo in the code, this mechanism broke the discuss application. Current behavior before PR: If the user put more than 200 emojis, the discuss app is broken Desired behavior after PR is merged: If the user put more than 200 emojis, it's working. -- I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr Forward-Port-Of: odoo/odoo#40701
Install Accounting, go to Configuration > Follow-up Levels. Create a new Follow-Up Action with low due days just to make testing easier and "Join open invoices" enabled. Go to Customers>"Follow-up Reports", select a follow up report with action "test", click on "Send Email" The email will be sent, without attachments. There should be a number of attachment equal to the number of due invoices. This append because the attachments are computed but are all filtered in the postprocess phase
Original PR description
Install Accounting, go to Configuration > Follow-up Levels. Create a new Follow-Up Action with low due days just to make testing easier and "Join open invoices" enabled. Go to Customers>"Follow-up Reports", select a follow up report with action "test", click on "Send Email" The email will be sent, without attachments. There should be a number of attachment equal to the number of due invoices. This append because the attachments are computed but are all filtered in the postprocess phase because either: * the attachment was created with a user id different from the one used to send the email * the attachment is not related to the current model (res.partner in this case) Removing the filter in this particular case solves the issue. opw-2116803 -- I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr Forward-Port-Of: odoo/odoo#40181
Forward-Port-Of: odoo/enterprise#6814
Original PR description
Forward-Port-Of: odoo/enterprise#6814