search

OCA / product-configurator

Advanced Product Configurator (from the original Pledra project https://github.com/pledra/odoo-product-configurator)
GNU Affero General Public License v3.0
33 stars 78 forks source link

[FIX] product_configurator: non-stored weight field is changed to a stored one #83

Closed RabbitJon-S73 closed 1 year ago

RabbitJon-S73 commented 1 year ago

(Reopened from https://github.com/OCA/product-configurator/pull/71) Correction of bug: original field weight is not-stored Fix issue: https://github.com/OCA/product-configurator/issues/66

OCA-git-bot commented 1 year ago

Hi @PCatinean, some modules you are maintaining are being modified, check this out!

RabbitJon-S73 commented 1 year ago

@DonatasV @andreampiovesana @dreispt

dreispt commented 1 year ago

/ocabot merge patch

OCA-git-bot commented 1 year ago

On my way to merge this fine PR! Prepared branch 14.0-ocabot-merge-pr-83-by-dreispt-bump-patch, awaiting test results.

OCA-git-bot commented 1 year ago

@dreispt your merge command was aborted due to failed check(s), which you can inspect on this commit of 14.0-ocabot-merge-pr-83-by-dreispt-bump-patch.

After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red.

RabbitJon-S73 commented 1 year ago

@DonatasV I don't know exactly how OCA-git-bot works, did something went wrong? Do I have to do anything else?

RabbitJon-S73 commented 1 year ago

@DonatasV @theangryangel i don't know how to fix the error during merge. Could you help me?

RabbitJon-S73 commented 1 year ago

@dreispt could we try again merge?

dreispt commented 1 year ago

/ocabot merge nobump

OCA-git-bot commented 1 year ago

This PR looks fantastic, let's merge it! Prepared branch 14.0-ocabot-merge-pr-83-by-dreispt-bump-nobump, awaiting test results.

OCA-git-bot commented 1 year ago

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