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.

Sign up for GitHub

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

Jump to bottom

[11.0][MIG] account_invoice_download #91

Closed
wants to merge 14 commits into from
Closed

[11.0][MIG] account_invoice_download #91

wants to merge 14 commits into from

Conversation

rousseldenis
Copy link
Sponsor

No description provided.

@rousseldenis
Copy link
Sponsor Author

@alexis-via

@rousseldenis rousseldenis mentioned this pull request Mar 8, 2019
Closed
1 task
@pedrobaeza pedrobaeza added this to the 11.0 milestone Mar 8, 2019
@rousseldenis
Copy link
Sponsor Author

WIP

@pedrobaeza pedrobaeza mentioned this pull request Mar 8, 2019
Closed
27 tasks
@tbaden
Copy link
Member

tbaden commented May 17, 2019

still WIP? @rousseldenis

@njeudy
Copy link
Collaborator

njeudy commented Sep 17, 2019

@rousseldenis any news on this ? need to port to 12.0 :)

@bosd
Copy link
Contributor

bosd commented Jan 22, 2021

ping @rousseldenis

bosd
bosd approved these changes Jan 22, 2021
View reviewed changes
Copy link
Contributor

@bosd bosd left a comment

Choose a reason for hiding this comment

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

LGTM, as far as I can functionally test it without backends 😉

@rousseldenis
Copy link
Sponsor Author

I think this one is ok if someone can test with backend. As I did this on freetime, I cannot more support on this

@github-actions
Copy link

github-actions bot commented Dec 5, 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 Dec 5, 2021
alexis-via and others added 12 commits January 9, 2022 10:44
@alexis-via @rousseldenis
Initial check-in of the modules account_invoice_download and account_…
8f18da9 
…invoice_download_weboob (still in development)
@alexis-via @rousseldenis
Add migration script
4fb7427
@alexis-via @rousseldenis
Display good error message when no invoices are downloaded via the
66e9d29 
credentials wizard

Don't rollback the update of last run if no invoices are downloaded
@alexis-via @rousseldenis
Add support for multiple invoice import configs in invoice import wizard
265c61f 
Update other modules accordingly
@alexis-via @rousseldenis
Add O2M field from import config to download config
54e505a 
Code update and cleanup
@alexis-via @rousseldenis
Finalize the development account_invoice_download
7a24783 
Update the account_invoice_download_weboob following the changes in account_invoice_download
Add README.rst for the 2 modules
@alexis-via @rousseldenis
Add support for additional module-specific weboob config parameters
811ca29 
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
@alexis-via @rousseldenis
Don't update last_run when download fails
1021ca1 
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)
@alexis-via @rousseldenis
Update headers to OCA standards
e14120a 
Update website and category keys in manifest
@alexis-via @rousseldenis
Improve log handling in ovh backend
e6d8e18 
Code cleanup
@alexis-via @rousseldenis
Add ability to run cron invoice download from admin for all companies…
042272f 
… 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
@oca-travis @rousseldenis
[UPD] Update account_invoice_download.pot
28aebcd
@rousseldenis rousseldenis force-pushed the 11.0-add_invoice_download branch 2 times, most recently from 0ac1c5b to 92d9049 Compare January 9, 2022 10:43
@rousseldenis rousseldenis force-pushed the 11.0-add_invoice_download branch 2 times, most recently from d8eae20 to 0eb7696 Compare January 9, 2022 15:52
@rousseldenis
Copy link
Sponsor Author

@tbaden @njeudy @alexis-via I've taken time to resuscitate this.

I've added tests for this module in a separate commit (can be easily forwarded).

Thanks

@rousseldenis rousseldenis mentioned this pull request Jan 9, 2022
Closed
2 tasks
@github-actions github-actions bot removed the stale PR/Issue without recent activity, it'll be soon closed automatically. label Jan 16, 2022
@github-actions
Copy link

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 May 22, 2022
@github-actions github-actions bot closed this Jun 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bosd bosd bosd approved these changes

Assignees
No one assigned
Labels
stale PR/Issue without recent activity, it'll be soon closed automatically.
Projects
None yet
Milestone
11.0
Development

Successfully merging this pull request may close these issues.

None yet
7 participants
@rousseldenis @tbaden @njeudy @bosd @pedrobaeza @alexis-via @oca-travis