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.

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

[16.0][MIG] base_business_document_import_phone #743

Merged

Conversation

alexis-via
Copy link
Contributor

We'll have to see if we update the match code to also use "mobile" in the pivot dict...

@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 Jul 23, 2023
@github-actions github-actions bot closed this Aug 27, 2023
@alexis-via alexis-via added no stale Use this label to prevent the automated stale action from closing this PR/Issue. and removed stale PR/Issue without recent activity, it'll be soon closed automatically. labels Oct 31, 2023
@alexis-via alexis-via reopened this Oct 31, 2023
alexis-via and others added 23 commits October 31, 2023 14:56
…import_phone for consistency

Add connector-telephony in oca_dependencies.txt
Add support for partner bank matching on invoice update (before, it was only supported on invoice creation)
[FIX] LINT

Use try/except when importing external libs
Remove self.ensure_one() that has nothing to do in an api.model method
Rename __openerp__.py to __manifest__.py and set installable to False
Also port all the modules that generate the XML documents: account_invoice_ubl, account_invoice_zugferd, purchase_order_ubl and sale_order_ubl
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)
… 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
Migrate base_business_document_import_phone from v10 to v12
Convert readme to the new format
Improve code for shipping addresse matching
Improve tests
Currently translated at 100.0% (1 of 1 strings)

Translation: edi-12.0/edi-12.0-base_business_document_import_phone
Translate-URL: https://translation.odoo-community.org/projects/edi-12-0/edi-12-0-base_business_document_import_phone/de/
Currently translated at 100.0% (1 of 1 strings)

Translation: edi-12.0/edi-12.0-base_business_document_import_phone
Translate-URL: https://translation.odoo-community.org/projects/edi-12-0/edi-12-0-base_business_document_import_phone/hr/
@alexis-via alexis-via force-pushed the 16-mig-base_business_document_import_phone branch from ffe2346 to 573c405 Compare October 31, 2023 13:57
@alexis-via alexis-via force-pushed the 16-mig-base_business_document_import_phone branch from 573c405 to 2f7ee5d Compare October 31, 2023 15:03
@alexis-via
Copy link
Contributor Author

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

What a great day to merge this nice PR. Let's do it!
Prepared branch 16.0-ocabot-merge-pr-743-by-alexis-via-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 2b221ea into OCA:16.0 Oct 31, 2023
7 checks passed
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at 7396c84. Thanks a lot for contributing to OCA. ❤️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
merged 🎉 no stale Use this label to prevent the automated stale action from closing this PR/Issue.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants