[13.0][MIG] account_invoice_import #727
Closed
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
Add a hook to inject parsed invoice from othe means than a file
Add a hook for an action on missing partner
A button is added on the Vendor Bill form to import invoices in serie This button can be accessed through the keyboard shortcut with Alt + M (or an other letter depending on the modules installed on the server
Some countries don't provide tax data in invoices, but invoices by themselves can have the price included tax. In this case, there will be attempt to create an adjustment line which leads to increasing total invoice amount and complete fail.
|
/ocabot migration account_invoice_import |
|
There's no issue in this repo with the title 'Migration to version 13.0' and the milestone 13.0, so not possible to add the comment. |
This was referenced Feb 21, 2023
|
@yvaucher You did some improvements on the module in this pr. Are there any commits which can be forward ported to V14? |
sonhd91
reviewed
May 9, 2023
| "website": "https://github.com/OCA/edi", | ||
| "depends": [ | ||
| "account", | ||
| "base_iban", |
There was a problem hiding this comment.
Suggested change
| "base_iban", | |
| "edi", | |
| "base_iban", |
|
@yvaucher Can you force push |
|
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
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.
Replaces #213