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

[FW][FIX] l10n_ar_account_withholding: certificados de retencion en envio de mails #992

Closed

Conversation

fw-bot-adhoc
Copy link
Contributor

Extendimos el método original de _compute_attachment_ids para poder agregar como adjuntos los certificados de retenciones en los mails de pagos.
Esto es posible hacerlo desde el botón de acción "send receipts by email" desde el pago individual, o seleccionando un pago desde la vista lista y luego clickeando el mismo botón de acción.

Según los docstrings del método original, en el template del mail sólo hay que mostrar los adjuntos cuando se trata de un sólo pago, y en caso de seleccionar más de un pago el template se muestra sin adjuntos y luego al momento de enviar el mail se deberían adjuntar los archivos correspondientes. Quedaría pendiente agregar esta posibilidad en caso de que se quiera implementar.
En v16 hay un bug ya que al seleccionar más de un pago dentro del template figuran los comprobantes de retención de un sólo pago y esos comprobantes son adjuntos a todos los mails de los pagos seleccionados. Es por eso que sólo dejé la funcionalidad para cuando se selecciona un solo pago.

Forward-Port-Of: #986

@roboadhoc
Copy link

@fw-bot-adhoc
Copy link
Contributor Author

@jue-adhoc @zaoral cherrypicking of pull request #986 failed.

stdout:

Auto-merging l10n_ar_tax/models/mail_compose_message.py
CONFLICT (content): Merge conflict in l10n_ar_tax/models/mail_compose_message.py

stderr:

13:00:22.783329 git.c:444               trace: built-in: git cherry-pick 871e106f42709c2b678099dfbc7f89693b525e65
error: could not apply 871e106f... [FIX] l10n_ar_account_withholding: certificados de retencion en envio de mails
hint: after resolving the conflicts, mark the corrected paths
hint: with 'git add <paths>' or 'git rm <paths>'
hint: and commit the result with 'git commit'
----------
status:

Either perform the forward-port manually (and push to this branch, proceeding as usual) or close this PR (maybe?).

In the former case, you may want to edit this PR message as well.

More info at https://github.com/odoo/odoo/wiki/Mergebot#forward-port

@jue-adhoc jue-adhoc force-pushed the 18.0-17.0-h-83108-jue-SWtj-fw branch from 2d9ffac to cb2bd41 Compare December 20, 2024 20:23
@jue-adhoc jue-adhoc closed this Dec 26, 2024
@jue-adhoc
Copy link
Contributor

Cierro este PR para poder tratarlo en gestión del cambio de v18, ya que cambiaron varias cosas de retenciones y requiere un análisis más profundo.

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