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

[MIG][14.0] partner_multi_company #411

Merged
merged 30 commits into from
Dec 20, 2022

Conversation

ilyasProgrammer
Copy link
Member

No description provided.

oihane and others added 29 commits November 23, 2022 20:22
This fix prevents errors when there are other tests that create partners
and base_suspend_security is still not ready. There can be still problems
if these partners are not created with SUPERUSER.
* Bump versions
* Rename manifest
* Rename openerp to odoo
* Implement base_multi_company

Add depends

Swap dependencies

Add base_suspend_security depend back & server-tools repo

Fix tests

Add test for partner._commercial_fields

Why not both?!
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: multi-company-11.0/multi-company-11.0-partner_multi_company
Translate-URL: https://translation.odoo-community.org/projects/multi-company-11-0/multi-company-11-0-partner_multi_company/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: multi-company-12.0/multi-company-12.0-partner_multi_company
Translate-URL: https://translation.odoo-community.org/projects/multi-company-12-0/multi-company-12-0-partner_multi_company/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: multi-company-12.0/multi-company-12.0-partner_multi_company
Translate-URL: https://translation.odoo-community.org/projects/multi-company-12-0/multi-company-12-0-partner_multi_company/
Currently translated at 100.0% (3 of 3 strings)

Translation: multi-company-13.0/multi-company-13.0-partner_multi_company
Translate-URL: https://translation.odoo-community.org/projects/multi-company-13-0/multi-company-13-0-partner_multi_company/es/
Copy link
Contributor

@francesco-ooops francesco-ooops left a comment

Choose a reason for hiding this comment

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

supersedes #349

functional ok!

Copy link

@elvise elvise left a comment

Choose a reason for hiding this comment

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

LGTM

@OCA-git-bot
Copy link
Contributor

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

@francesco-ooops
Copy link
Contributor

@OCA/intercompany-maintainers when you can :)

@francesco-ooops
Copy link
Contributor

@OCA/intercompany-maintainers can this be merged?

@francesco-ooops
Copy link
Contributor

@OCA/intercompany-maintainers migration review please :)

@matteoopenf
Copy link
Contributor

@pedrobaeza is possibible to merge?

@pedrobaeza pedrobaeza changed the title [MIG][14.0] partner multi company [MIG][14.0] partner_multi_company Dec 20, 2022
@pedrobaeza
Copy link
Member

/ocabot migration partner_multi_company
/ocabot merge nobump

@OCA-git-bot OCA-git-bot added this to the 14.0 milestone Dec 20, 2022
@OCA-git-bot OCA-git-bot mentioned this pull request Dec 20, 2022
15 tasks
@OCA-git-bot
Copy link
Contributor

The migration issue (#245) has been updated to reference the current pull request.
however, a previous pull request was referenced : #349.
Perhaps you should check that there is no duplicate work.
CC : @JasminSForgeFlow

@OCA-git-bot
Copy link
Contributor

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 14.0-ocabot-merge-pr-411-by-pedrobaeza-bump-nobump, awaiting test results.

@OCA-git-bot
Copy link
Contributor

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

@matteoopenf
Copy link
Contributor

/ocabot migration partner_multi_company /ocabot merge nobump

thank you

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet