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

[11.0][MIG] account_invoice_import_invoice2data #76

Closed

Conversation

tarteo
Copy link
Member

@tarteo tarteo commented Oct 31, 2018

No description provided.

alexis-via and others added 29 commits October 31, 2018 09:34
…edicated module account_invoice_import_invoice2data

Update README.rst and headers to latest OCA conventions.
Better key names in the parsed_inv dict
parsed_inv doesn't need to be JSON serializable anymore (small drawback: the invoice is parsed a second time on the second step... but the second step is rarely used)
Move code from account_invoice_import_invoice2data to account_invoice_import
Update REAME and some interface strings about UBL being an ISO standard
Small code changes
…voice dict, cleaner organisation)

Code refactoring: move code in base_business_document_import, factorise code for tax matching (it was duplicated in UBL and ZUGFeRD)
Now support PDF with embedded UBL XML file
Enable unittests on account_invoice_import_ubl
More absolute xpath in account_invoice_import_ubl instead of relative xpath

WARNING: these are big changes, I may have broken a few details
…te dir and the built-in templates

Also allow to use only a local template dir
README updated to explain how to configure all this
Special thanks to Sébastien Beau for his help to achieve this
Add support for partner bank matching on invoice update (before, it was only supported on invoice creation)
[FIX] LINT

Use try/except when importing external libs
Remove self.ensure_one() that has nothing to do in an api.model method
…a recent version of pdftotext on travis's ubuntu 12.04 images
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
… module

Fix spelling mistake and other remarks on README by Tarteo
* Update to work with latest version of invoice2data

* Add requirements.txt file
Update the account_invoice_download_weboob following the changes in account_invoice_download
Add README.rst for the 2 modules
Update text displayed in the invoice import wizard and make list of
supported formats modular (like in bank statement import)
@pedrobaeza pedrobaeza added this to the 11.0 milestone Oct 31, 2018
@pedrobaeza pedrobaeza mentioned this pull request Oct 31, 2018
27 tasks
@astirpe
Copy link
Member

astirpe commented Oct 31, 2018

From the logs of Runbot:

2018-10-31 08:43:09,622 166 WARNING openerp_test odoo.models: res.partner.create() includes unknown fields: invoice_import_id

@bosd
Copy link
Contributor

bosd commented Feb 2, 2019

newby question.
Now the pdftotext is in the external dependencies. I creates an error on runbot since it is not installed.
How to install this on the runbot? To do further testing?

@tarteo
Copy link
Member Author

tarteo commented Feb 5, 2019

@bosd It's need to be added to https://github.com/OCA/edi/blob/11.0/.travis.yml I will do it.

@rousseldenis
Copy link
Sponsor

@tarteo Will you fix it ?

I'll maybe migrate this to 12

@tarteo
Copy link
Member Author

tarteo commented Mar 22, 2019

@rousseldenis Yes I'll try

@tarteo
Copy link
Member Author

tarteo commented Mar 22, 2019

@rousseldenis It works, but I can't get travis to work.

@rousseldenis
Copy link
Sponsor

@alexis-via There is a problem to fetch 'https://akretion.com/pdftotext-3.04'

Do you have any clue ?

@extrememicro
Copy link

@tarteo

The command "wget -P /tmp http://public.akretion.com/pdftotext-3.04" failed and exited with 8 during .

@alexis-via
Copy link
Contributor

@extrememicro the URL http://public.akretion.com/pdftotext-3.04 is working again, thanks to the work of my collegue @sebastienbeau

@feketemihai
Copy link
Member

@tarteo Can you rebase, to see if it's working?

@github-actions
Copy link

github-actions bot commented Nov 7, 2021

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days.
If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

@github-actions github-actions bot added the stale PR/Issue without recent activity, it'll be soon closed automatically. label Nov 7, 2021
@github-actions github-actions bot closed this Dec 12, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs review stale PR/Issue without recent activity, it'll be soon closed automatically.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet