Skip to content

[15.0][MIG] product_attribute_value_menu#1128

Merged
OCA-git-bot merged 14 commits into
OCA:15.0from
ForgeFlow:15.0-mig-product_attribute_value_menu
Sep 2, 2022
Merged

[15.0][MIG] product_attribute_value_menu#1128
OCA-git-bot merged 14 commits into
OCA:15.0from
ForgeFlow:15.0-mig-product_attribute_value_menu

Conversation

@JasminSForgeFlow
Copy link
Copy Markdown
Contributor

Standard Migration

@ForgeFlow

Copy link
Copy Markdown

@DavidJForgeFlow DavidJForgeFlow 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 👍 also code LGTM

Copy link
Copy Markdown

@JuanyDForgeflow JuanyDForgeflow left a comment

Choose a reason for hiding this comment

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

LGTM!

Copy link
Copy Markdown
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 review

@OCA-git-bot
Copy link
Copy Markdown
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). 🤖

@JordiMForgeFlow
Copy link
Copy Markdown
Contributor

Hi @JasminSForgeFlow

Could you take into consideration also #1132? :)

@JasminSForgeFlow JasminSForgeFlow force-pushed the 15.0-mig-product_attribute_value_menu branch from f0d6a67 to 6452ed8 Compare August 22, 2022 06:23
@JasminSForgeFlow
Copy link
Copy Markdown
Contributor Author

Hi @JasminSForgeFlow

Could you take into consideration also #1132? :)

done, thanks

@JordiMForgeFlow
Copy link
Copy Markdown
Contributor

Hi @JasminSForgeFlow

Could you apply again the changes in #1132? small changes were added 4 days ago. The PR is now merged :)

@JasminSForgeFlow JasminSForgeFlow force-pushed the 15.0-mig-product_attribute_value_menu branch from 6452ed8 to 709ca0f Compare August 30, 2022 11:42
@JasminSForgeFlow
Copy link
Copy Markdown
Contributor Author

Hi @JasminSForgeFlow

Could you apply again the changes in #1132? small changes were added 4 days ago. The PR is now merged :)

Done, Thanks

@JordiBForgeFlow
Copy link
Copy Markdown
Member

/ocabot migration product_attribute_value_menu
/ocabot merge nobump

@OCA-git-bot OCA-git-bot added this to the 15.0 milestone Sep 2, 2022
@OCA-git-bot
Copy link
Copy Markdown
Contributor

On my way to merge this fine PR!
Prepared branch 15.0-ocabot-merge-pr-1128-by-JordiBForgeFlow-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot mentioned this pull request Sep 2, 2022
65 tasks
@OCA-git-bot OCA-git-bot merged commit 82c089c into OCA:15.0 Sep 2, 2022
@OCA-git-bot
Copy link
Copy Markdown
Contributor

Congratulations, your PR was merged at 0202bdb. 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

Projects

None yet

Development

Successfully merging this pull request may close these issues.