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] account: warning when sending invoice by email #33753

Conversation

jpp-odoo
Copy link
Contributor

Before this commit, when sending multiples invoices by email in batch,
if one of these invoices was for a customer without an email address,
these wasn't sent and the user didn't have any feedback about this.

Now, before the user send the emails a warning is shown with the list of
invoices that can't be sent by email.

opw-2002350

Before this commit, when sending multiples invoices by email in batch,
if one of these invoices was for a customer without an email address,
these wasn't sent and the user didn't have any feedback about this.

Now, before the user send the emails a warning is shown with the list of
invoices that can't be sent by email.

opw-2002350
@jpp-odoo jpp-odoo requested a review from nim-odoo May 29, 2019 12:44
@C3POdoo C3POdoo added the OE the report is linked to a support ticket (opw-...) label May 29, 2019
@robodoo robodoo added the CI 🤖 Robodoo has seen passing statuses label May 29, 2019
Copy link
Contributor

@nim-odoo nim-odoo left a comment

Choose a reason for hiding this comment

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

robodoo delegate+

@jpp-odoo
Copy link
Contributor Author

jpp-odoo commented Jun 3, 2019

robodoo r+

@robodoo
Copy link
Contributor

robodoo commented Jun 3, 2019

Merged, thanks!

@jpp-odoo jpp-odoo deleted the 12.0-opw-2002350-warning_send_invoice_batch-jpp branch June 3, 2019 06:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI 🤖 Robodoo has seen passing statuses OE the report is linked to a support ticket (opw-...)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants