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

[ADD][16.0] account_invoice_transmit #1715

Open
wants to merge 6 commits into
base: 16.0
Choose a base branch
from

Conversation

jbaudoux
Copy link

@jbaudoux jbaudoux commented Apr 26, 2024

Implements the sending of invoices in background as defined by the transmit method.
Replaces the standard "send and print" odoo action on list view with this new wizard.
Report to print is generated in background.

image

Note: another module will follow for sending by peppol

cc @lmignon @rousseldenis @simahawk

@jbaudoux jbaudoux force-pushed the 16-account_invoice_transmit branch 5 times, most recently from b5216bb to b1dac1e Compare April 28, 2024 09:30
@jbaudoux jbaudoux marked this pull request as ready for review April 28, 2024 09:32
@jbaudoux jbaudoux changed the title [WIP][ADD][16.0] account_invoice_transmit [ADD][16.0] account_invoice_transmit Apr 28, 2024
@jbaudoux jbaudoux force-pushed the 16-account_invoice_transmit branch 2 times, most recently from 742e559 to f08dd78 Compare April 29, 2024 13:25
@@ -0,0 +1,3 @@
* Jacques-Etienne Baudoux (BCIM) <[email protected]>
* Camptocamp
* Acsone SA
Copy link
Contributor

Choose a reason for hiding this comment

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

@jbaudoux If you can put me here and ACSONE as author too (for the tests essentially) ? Thanks

@jbaudoux jbaudoux force-pushed the 16-account_invoice_transmit branch 3 times, most recently from d4d1208 to 954dfc6 Compare May 6, 2024 10:09
@jbaudoux jbaudoux force-pushed the 16-account_invoice_transmit branch from 954dfc6 to 125a219 Compare May 6, 2024 10:13
Comment on lines +17 to +19
document = fields.Binary(
comodel_name="ir.attachment", attachment=True, readonly=True
)
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
document = fields.Binary(
comodel_name="ir.attachment", attachment=True, readonly=True
)
document = fields.Binary(attachment=True, readonly=True)

Copy link
Contributor

@lmignon lmignon left a comment

Choose a reason for hiding this comment

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

LGTM (Code review only)

@jbaudoux jbaudoux force-pushed the 16-account_invoice_transmit branch 2 times, most recently from 6aa3f3f to 5783aca Compare May 21, 2024 11:05
prevent 2 sending jobs running in parallel to send the same invoice
@jbaudoux jbaudoux force-pushed the 16-account_invoice_transmit branch from 5783aca to 5e91029 Compare May 21, 2024 12:43
rousseldenis and others added 2 commits October 29, 2024 12:44
As the number of invoices can be huge, the job can fail in timeout.
[IMP] account_invoice_transmit: Use a separate job for sending emails
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.

3 participants