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

[13.0][MIG] account_invoice_ubl #156

Merged
merged 38 commits into from
May 13, 2020

Conversation

astirpe
Copy link
Member

@astirpe astirpe commented Dec 13, 2019

@OCA-git-bot OCA-git-bot mentioned this pull request Dec 13, 2019
11 tasks
Copy link

@CasVissers-360ERP CasVissers-360ERP left a comment

Choose a reason for hiding this comment

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

👍 Functional review.
image

@CasVissers-360ERP
Copy link

@astirpe
Sorry did find something after-all :( When printing invoices (PDF) I get an empty PDF file.

If I comment this line: pdf_content = invoice.embed_ubl_xml_in_pdf(pdf_content)
The PDF starts working again..

@astirpe
Copy link
Member Author

astirpe commented Dec 24, 2019

@CasVissers the empty PDF should be now fixed by fdaaab3 on module base_ubl (#154). Could you check?

@CasVissers-360ERP
Copy link

@astirpe Yes I can confirm it works now.

@astirpe astirpe force-pushed the 13_mig_account_invoice_ubl branch from e91b3e6 to 92f9f5d Compare March 3, 2020 12:31
alexis-via and others added 23 commits March 26, 2020 11:26
Print numbers with the approriate number of digits in the XML
Add support for partner bank matching on invoice update (before, it was only supported on invoice creation)
Rename __openerp__.py to __manifest__.py and set installable to False
Also port all the modules that generate the XML documents: account_invoice_ubl, account_invoice_zugferd, purchase_order_ubl and sale_order_ubl
Rename account_invoice_import_zugferd to account_invoice_import_factur-x
Add module to support py3o reporting engine:
Add active buttons on invoice import config
… module

Fix spelling mistake and other remarks on README by Tarteo
Add ubl invoice generation option in accounting config page
…xist in v10) in account_invoice_factur-x and account_invoice_ubl
Add script mass_invoice_import.py
Code cleanup
* sale_order_ubl + purchase_order_ubl: restore get_pdf() in tests using HttpCase

* Restore get_pdf() in tests of account_invoice_factur-x and account_invoice_ubl modules

* Update oca_dependencies.txt
… embedding in PDF between account_invoice_factur-x and account_invoice_ubl
Enable delivery block in UBL
@aisopuro
Copy link

This seems to have been sitting still with green lights for a while. @CasVissers do you know of any other maintainers of this repo who could take a look at this so it could be merged?

@CasVissers-360ERP
Copy link

@alexis-via

Copy link
Member

@jgrandguillaume jgrandguillaume left a comment

Choose a reason for hiding this comment

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

LGTM thanks

@OCA-git-bot
Copy link
Contributor

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

@alexis-via alexis-via merged commit 55a4c96 into OCA:13.0 May 13, 2020
@astirpe astirpe deleted the 13_mig_account_invoice_ubl branch May 14, 2020 06:07
@alexis-via
Copy link
Contributor

I found 2 bugs in this PR:

  1. it generates UBL XML even when invoice.company_id.xml_format_in_pdf_invoice == facturx !
  2. the button to generate UBL XML on invoice is shown in draft state, but it crashes
    I'll fix it in my PR to port Factur-X to v13.

@CasVissers-360ERP
Copy link

@alexis-via

I found 2 bugs in this PR:

  1. it generates UBL XML even when invoice.company_id.xml_format_in_pdf_invoice == facturx !
  2. the button to generate UBL XML on invoice is shown in draft state, but it crashes
    I'll fix it in my PR to port Factur-X to v13.

Did you fix point 2 somewhere I can check?

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.

None yet

9 participants