-
-
Notifications
You must be signed in to change notification settings - Fork 296
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] sale_order_import_ubl #61
Conversation
fbd49f6
to
07437cc
Compare
@tarteo I cherry-picked the latest commit from V10.0 and rebased the branch. I hope is ok for you. |
Coverage increased (+8.6%) to 73.36% when pulling 5729449bc5e6d800515bb8a3a991e53478883750 on tarteo:11.0-mig-sale_order_import_ubl into 6994600 on OCA:11.0. |
Coverage increased (+6.009%) to 61.526% when pulling 7e3c02e0d2db31d76783a56811e5ffae0639735d on tarteo:11.0-mig-sale_order_import_ubl into 6bbb7a3 on OCA:11.0. |
5729449
to
7e3c02e
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Works fine
@tarteo could you rebase and resolve the conflict? |
…, sale_order_import_ubl, sale_order_import_csv, sale_order_ubl, sale_commercial_partner
Add and update README
Add support for partner bank matching on invoice update (before, it was only supported on invoice creation)
Rename __openerp__.py to __manifest__.py and set installable to False
… module Fix spelling mistake and other remarks on README by Tarteo
7e3c02e
to
05b6e1c
Compare
…iness document is imported in the right company
[TO REMOVE] Add temporary dependency on open PR Add readme fragments
05b6e1c
to
6a2065b
Compare
@astirpe 👍 done. |
@pedrobaeza this PR is the last one regarding the UBL still pending. Is it ok to merge it? |
Yes, merging, I'm having fully confidence in your team work 😃 |
Depends on: