Skip to content

Conversation

gpozo-gisce
Copy link

@gpozo-gisce gpozo-gisce commented Feb 19, 2025

Objetivos

  • When you send a report attachment with poweremail now you can attach reports of other models differents from the template's reference model.

Comportamiento antiguo

  • N/A

Comportamiento nuevo

Allow to attach to the mails, another data models with a regex like: ('<model>', <conditions>)

image

Afectaciones / Migración de datos

Relacionado

Copy link

github-actions bot commented Feb 19, 2025

PY2TestsResults

  2 files    2 suites   9m 4s ⏱️
104 tests 104 ✅ 0 💤 0 ❌
105 runs  105 ✅ 0 💤 0 ❌

Results for commit 4dbee03.

♻️ This comment has been updated with latest results.

Copy link

github-actions bot commented Feb 19, 2025

PY3TestsResults

  2 files    2 suites   9m 20s ⏱️
104 tests 102 ✅ 0 💤 2 ❌
105 runs  103 ✅ 0 💤 2 ❌

For more details on these failures, see this check.

Results for commit 4dbee03.

♻️ This comment has been updated with latest results.

BREAKING CHANGE: This will make a duplicatation from the 1r commit from PR #85
joanperez1
joanperez1 previously approved these changes Jul 11, 2025
Copy link
Contributor

@joanperez1 joanperez1 left a comment

Choose a reason for hiding this comment

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

HVB

@gpozo-gisce gpozo-gisce changed the title Make attachments of other models in poweremail.template Make attachments of other models in poweremail.template Sep 24, 2025
<openerp>
<data>
<record model="ir.ui.view" id="poweremail_template_attachment_form">
<record model="ir.ui.view" id="poweremail_template_attachment_form">
Copy link
Member

Choose a reason for hiding this comment

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

pep8

@gpozo-gisce gpozo-gisce added the wip Work In Progress or being tested label Oct 8, 2025
@gpozo-gisce gpozo-gisce requested a review from lcbautista October 9, 2025 14:14
@gpozo-gisce gpozo-gisce removed the wip Work In Progress or being tested label Oct 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants