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.

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

Make attachments of other models #85

Draft
wants to merge 3 commits into
base: v5_backport
Choose a base branch
from

Conversation

destanyol
Copy link
Contributor

@destanyol destanyol commented Mar 2, 2022

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: ${get('<data_model>').search(cursor, uid, [(condition1), (condition2)])}

New column:
image

Afectaciones / Migración de datos

Relacionado

Checklist

  • Test code
  • Documentation (link to PowERP docs repo)
  • Si se modifica alguna vista poner una captura indicando qué se modifica
  • Si se modifica un report adjuntar el report
  • Migration data

@destanyol destanyol force-pushed the report_id_customizable branch from 66ebd52 to ccbbe8a Compare March 2, 2022 14:29
@polsala polsala added this to the v23.12 milestone Oct 4, 2023
@gpozo-gisce gpozo-gisce added the wip Work In Progress or being tested label Feb 18, 2025
@gpozo-gisce gpozo-gisce self-assigned this Feb 18, 2025
@gpozo-gisce gpozo-gisce marked this pull request as draft February 19, 2025 10:35
Copy link

PY3TestsResults

 2 files  ±0   2 suites  ±0   1m 44s ⏱️ -1s
86 tests ±0  86 ✅ ±0  0 💤 ±0  0 ❌ ±0 
87 runs  ±0  87 ✅ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit b818dd5. ± Comparison against base commit 7c8b0d1.

Copy link

PY2TestsResults

 2 files  ±0   2 suites  ±0   1m 49s ⏱️ +6s
86 tests ±0  86 ✅ ±0  0 💤 ±0  0 ❌ ±0 
87 runs  ±0  87 ✅ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit b818dd5. ± Comparison against base commit 7c8b0d1.

gpozo-gisce added a commit that referenced this pull request Feb 20, 2025
BREAKING CHANGE: This will make a duplicatation from the 1r commit from PR #85
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement to be merged wip Work In Progress or being tested
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants