-
Notifications
You must be signed in to change notification settings - Fork 199
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
Conversation
""" 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] |
There was a problem hiding this comment.
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
a2cae23
to
b0ee5df
Compare
b0ee5df
to
871e106
Compare
@roboadhoc r+ |
… de mails closes #986 Signed-off-by: Katherine Zaoral <[email protected]>
@jue-adhoc @zaoral this pull request has forward-port PRs awaiting action (not merged or closed): |
6 similar comments
@jue-adhoc @zaoral this pull request has forward-port PRs awaiting action (not merged or closed): |
@jue-adhoc @zaoral this pull request has forward-port PRs awaiting action (not merged or closed): |
@jue-adhoc @zaoral this pull request has forward-port PRs awaiting action (not merged or closed): |
@jue-adhoc @zaoral this pull request has forward-port PRs awaiting action (not merged or closed): |
@jue-adhoc @zaoral this pull request has forward-port PRs awaiting action (not merged or closed): |
@jue-adhoc @zaoral this pull request has forward-port PRs awaiting action (not merged or closed): |
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.