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

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

Closed
wants to merge 1 commit into from

Conversation

jue-adhoc
Copy link
Contributor

@jue-adhoc jue-adhoc commented Dec 10, 2024

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.

@roboadhoc
Copy link

""" Extendemos el método original para que se pueda previsualizar en el envío de mailsde pagos el/los archivo de retenciones. """
for composer in self:
super()._compute_attachment_ids()
res_ids = composer._evaluate_res_ids() or [0]
Copy link
Contributor Author

@jue-adhoc jue-adhoc Dec 10, 2024

Choose a reason for hiding this comment

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

Ver comentario del PR sobre caso de envío masivo de emails, ese es el caso en que res_ids va a ser mayor a 1, y habría que ver de implementar la funcionalidad de que los attachments se agreguen a cada email en el momento de enviarse, pero dentro del composer no debe mostarse ninguno

@jue-adhoc jue-adhoc force-pushed the 17.0-h-83108-jue branch 9 times, most recently from a2cae23 to b0ee5df Compare December 12, 2024 18:23
@zaoral
Copy link
Contributor

zaoral commented Dec 16, 2024

@roboadhoc r+

@fw-bot-adhoc
Copy link
Contributor

@jue-adhoc @zaoral this pull request has forward-port PRs awaiting action (not merged or closed):

6 similar comments
@fw-bot-adhoc
Copy link
Contributor

@jue-adhoc @zaoral this pull request has forward-port PRs awaiting action (not merged or closed):

@fw-bot-adhoc
Copy link
Contributor

@jue-adhoc @zaoral this pull request has forward-port PRs awaiting action (not merged or closed):

@fw-bot-adhoc
Copy link
Contributor

@jue-adhoc @zaoral this pull request has forward-port PRs awaiting action (not merged or closed):

@fw-bot-adhoc
Copy link
Contributor

@jue-adhoc @zaoral this pull request has forward-port PRs awaiting action (not merged or closed):

@fw-bot-adhoc
Copy link
Contributor

@jue-adhoc @zaoral this pull request has forward-port PRs awaiting action (not merged or closed):

@fw-bot-adhoc
Copy link
Contributor

@jue-adhoc @zaoral this pull request has forward-port PRs awaiting action (not merged or closed):

@fw-bot-adhoc fw-bot-adhoc deleted the 17.0-h-83108-jue branch December 30, 2024 13:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants