[16.0][MIG] account_invoice_import #753
Draft
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
FIX raise error when we don't identify a partner Update README
…+ instructions OCR
Rename test/pdf/ to test/invoices/ to host XML invoices too
… total untaxed (depends on invoice layout)
PEP8 stuff
…invoice_zugferd (see README.rst for more info) Move the ZUGFeRD/CII-specific code of account_invoice_import to account_invoice_import_zugferd
Disable test of invoice2data, because this lib is difficult to install
Add support for UBL refunds If you have an UBL refund that I could use to test, please contact me !
Convert YAML test to unittest
…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
Adapt module account_invoice_import_ubl to use the new base_ubl module Small fixes
Add unitests in base_business_document_import Small code enhancements/simplifications
Code cleanup/minor changes
Currently translated at 94.3% (100 of 106 strings) Translation: edi-14.0/edi-14.0-account_invoice_import Translate-URL: https://translation.odoo-community.org/projects/edi-14-0/edi-14-0-account_invoice_import/fr/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: edi-14.0/edi-14.0-account_invoice_import Translate-URL: https://translation.odoo-community.org/projects/edi-14-0/edi-14-0-account_invoice_import/
The method get_invoice_line_account() was removed from the "account" module in odoo v13. So we now use the underlying method get_product_accounts() Fixes bug OCA#572
First draft migration, to have a first working version. Many more changes are planned.
|
Think I wrongly approve this (mistake with account_invoice_download) sorry ! |
|
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. |
github-actions
bot
added
the
stale
gurneyalex
added
no stale
|
Hello @alexis-via what is the status of this pull request? |
This was referenced Feb 20, 2024
|
#938 is the same PR, rebased, with the tests fixed. |
|
@alexis-via ping. Shall we keep this or move on w/ #938 ? |
Add support for discount on lines in the multi-line create scenario Fix force tax scenario and simplify the code
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This is a draft PR. Many more changes are planned.