Skip to content

Conversation

@simahawk
Copy link
Contributor

@simahawk simahawk commented Apr 1, 2025

Replaces #441

sebastienbeau and others added 30 commits April 1, 2025 13:27
This PR fixes the media displayed on the product variant form in the media
tab.

The change on the depends force the recomputation when an attibute value
is changed on the product.media.relation.

The other change fixes the attributes value that are tested to assign
a media on a variant.
It is the attribute set on the product variant that should be used and
not all attribute values assigned to the product template.

Forward port from:

* OCA#116
@simahawk
Copy link
Contributor Author

simahawk commented Apr 1, 2025

Former PR already approved
/ocabot migration storage_media_product
/ocabot merge nobump

@OCA-git-bot OCA-git-bot added this to the 18.0 milestone Apr 1, 2025
@OCA-git-bot
Copy link
Contributor

On my way to merge this fine PR!
Prepared branch 18.0-ocabot-merge-pr-465-by-simahawk-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot mentioned this pull request Apr 1, 2025
26 tasks
@OCA-git-bot OCA-git-bot merged commit d4408e7 into OCA:18.0 Apr 1, 2025
5 of 7 checks passed
@OCA-git-bot
Copy link
Contributor

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