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

[13.0][MIG] mrp_bom_structure_report_level_1 #68

Merged

Conversation

JoanSForgeFlow
Copy link

@JoanSForgeFlow JoanSForgeFlow commented Dec 23, 2019

Standard mrp_bom_structure_report_level_1 migration to v13.

cc~ @ForgeFlow @MiquelRForgeFlow

@JoanSForgeFlow JoanSForgeFlow force-pushed the 13.0-mig-mrp_bom_structure_report_level_1 branch from a0e5204 to ebff3f5 Compare December 23, 2019 12:37
@HviorForgeFlow
Copy link
Member

Split commit into pre-commit and migration one

@JoanSForgeFlow JoanSForgeFlow force-pushed the 13.0-mig-mrp_bom_structure_report_level_1 branch from ebff3f5 to 8a7ffee Compare January 8, 2020 09:33
Copy link

@AdriaGForgeFlow AdriaGForgeFlow left a comment

Choose a reason for hiding this comment

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

Small comment after code review

@JoanSForgeFlow JoanSForgeFlow force-pushed the 13.0-mig-mrp_bom_structure_report_level_1 branch from 8a7ffee to 4be32b1 Compare January 8, 2020 13:04
Copy link

@AdriaGForgeFlow AdriaGForgeFlow left a comment

Choose a reason for hiding this comment

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

Tested on local 👍

Copy link
Contributor

@LoisRForgeFlow LoisRForgeFlow left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@LoisRForgeFlow
Copy link
Contributor

/ocabot merge

@OCA-git-bot
Copy link
Contributor

What a great day to merge this nice PR. Let's do it!
Prepared branch 13.0-ocabot-merge-pr-68-by-LoisRForgeFlow-bump-no, awaiting test results.

OCA-git-bot added a commit that referenced this pull request Jan 13, 2020
Signed-off-by LoisRForgeFlow
@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). 🤖

@OCA-git-bot OCA-git-bot merged commit 4be32b1 into OCA:13.0 Jan 13, 2020
@OCA-git-bot
Copy link
Contributor

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

@LoisRForgeFlow LoisRForgeFlow deleted the 13.0-mig-mrp_bom_structure_report_level_1 branch January 13, 2020 11:08
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

9 participants