Skip to content

[14.0][IMP] product_attribute_value_menu: merge module product_attribute_value_menu from V13#1132

Merged
OCA-git-bot merged 2 commits into
OCA:14.0from
ForgeFlow:14.0-imp-product_attribute_value_menu
Aug 30, 2022
Merged

[14.0][IMP] product_attribute_value_menu: merge module product_attribute_value_menu from V13#1132
OCA-git-bot merged 2 commits into
OCA:14.0from
ForgeFlow:14.0-imp-product_attribute_value_menu

Conversation

@JordiMForgeFlow
Copy link
Copy Markdown
Contributor

As discussed and agreed with the modules authors in #1079, due to two modules having the same name (one is in V13 and one is in V14) we are merging their features in the same module.

Comment thread product_attribute_value_menu/models/product_attribute_value.py Outdated
@JordiMForgeFlow JordiMForgeFlow force-pushed the 14.0-imp-product_attribute_value_menu branch from 5c748cc to 90538ac Compare August 12, 2022 07:15
Copy link
Copy Markdown
Contributor

@GuillemCForgeFlow GuillemCForgeFlow left a comment

Choose a reason for hiding this comment

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

LGTM 👍

Comment thread product_attribute_value_menu/README.rst Outdated
Copy link
Copy Markdown
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.

@JordiMForgeFlow just a minor change, functional review ok! 👍

@JordiMForgeFlow JordiMForgeFlow force-pushed the 14.0-imp-product_attribute_value_menu branch from 90538ac to 9f3cc31 Compare August 26, 2022 05:45
@JordiMForgeFlow JordiMForgeFlow requested review from GuillemCForgeFlow and francesco-ooops and removed request for francesco-ooops August 29, 2022 06:39
Copy link
Copy Markdown
Contributor

@GuillemCForgeFlow GuillemCForgeFlow 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 👍

@francesco-ooops
Copy link
Copy Markdown
Contributor

@pedrobaeza could you check? fixes this issue

@pedrobaeza pedrobaeza added this to the 14.0 milestone Aug 30, 2022
@pedrobaeza
Copy link
Copy Markdown
Member

/ocabot merge major

@OCA-git-bot
Copy link
Copy Markdown
Contributor

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

@OCA-git-bot OCA-git-bot merged commit a5d592f into OCA:14.0 Aug 30, 2022
@OCA-git-bot
Copy link
Copy Markdown
Contributor

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

5 participants