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] Migration of sales_team_operating unit #613

Merged
merged 27 commits into from
Aug 2, 2023

Conversation

alan196
Copy link
Contributor

@alan196 alan196 commented Jul 27, 2023

This PR supersedes #592

I have removed the domain of company_id in operating unit, this is defined by the ir.rule of module operating_unit, so it's not needed.

This raises errors as in V16 the field company_id belongs to a group, and give error on installation.

@alan196 alan196 mentioned this pull request Jul 27, 2023
10 tasks
Copy link
Contributor

@AaronHForgeFlow AaronHForgeFlow left a comment

Choose a reason for hiding this comment

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

Functional LG!

@alan196
Copy link
Contributor Author

alan196 commented Aug 2, 2023

@EduardoJarsa @YahairaCVJarsa could you help me to review this PR 🙏

EduardoJarsa

This comment was marked as duplicate.

EduardoJarsa

This comment was marked as duplicate.

EduardoJarsa

This comment was marked as duplicate.

Copy link

@EduardoJarsa EduardoJarsa left a comment

Choose a reason for hiding this comment

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

Functional review, everything looks working

@AaronHForgeFlow
Copy link
Contributor

/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-613-by-AaronHForgeFlow-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 46a7b9f into OCA:16.0 Aug 2, 2023
@OCA-git-bot
Copy link
Contributor

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

@alan196 alan196 deleted the 16.0-mig-sales_team_operating_unit branch August 2, 2023 17:37
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.