-
-
Notifications
You must be signed in to change notification settings - Fork 307
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
[10.0] account_invoice_download + weboob and ovh backends (also add support for multi-invoice import config and mail gateway) #30
Merged
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
Warning: this PR requires some fixes that are available on this PR #22 |
alexis-via
force-pushed
the
10-download
branch
from
August 14, 2018 13:14
472250b
to
bc86eae
Compare
alexis-via
force-pushed
the
10-download
branch
from
August 18, 2018 23:08
57c290e
to
deb5781
Compare
alexis-via
changed the title
[10.0] account_invoice_download and account_invoice_download_weboob
[10.0] account_invoice_download + weboob and ovh backends
Aug 21, 2018
purchase_ok is defined in the purchase module, so we can't use it because we don't depend on it Add debug logs
Skip invoices with amount 0 Add debug messages
Add OVH demo products Add logs
Re-organise code for better perfs
…invoice_download_weboob (still in development)
credentials wizard Don't rollback the update of last run if no invoices are downloaded
Update other modules accordingly
Code update and cleanup
Update the account_invoice_download_weboob following the changes in account_invoice_download Add README.rst for the 2 modules
Add weboob module management Improve log handling (explicit error when wrong pwd) account_invoice_import: by default, set tax to default tax of account when invoice_line_method = 1line_no_product
When using auto invoice download, the cron is executed by default by admin user, which is above record rules: only handle download config in the company of the user and improve matching by always selecting obj in the same company (or company = False)
Update website and category keys in manifest
Port to OVH API (instead of SoAPI)
alexis-via
force-pushed
the
10-download
branch
from
August 21, 2018 22:38
0fdd39c
to
3942912
Compare
I merged with the project https://github.com/akretion/ovh-supplier-invoice/, renamed the module ovh_supplier_invoice to account_invoice_download_ovh and adapted the code to port to v10, use OVH API instead of SoAPI and, more importantly, start to use the account_invoice_download framework. |
alexis-via
changed the title
[10.0] account_invoice_download + weboob and ovh backends
[10.0] account_invoice_download + weboob and ovh backends (also add support for multi-invoice import config)
Aug 21, 2018
Code cleanup
Update text displayed in the invoice import wizard and make list of supported formats modular (like in bank statement import)
alexis-via
changed the title
[10.0] account_invoice_download + weboob and ovh backends (also add support for multi-invoice import config)
[10.0] account_invoice_download + weboob and ovh backends (also add support for multi-invoice import config and mail gateway)
Aug 22, 2018
… in multi-company environnement Fix my previous commit on sequence field on invoice import config (caused a crash when creating a new record) Usability improvements on download config
…iness document is imported in the right company
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.
No description provided.