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

fix(core): Fix custom field relation for ProductVariant when value is null #2727

Conversation

andriinuts
Copy link
Contributor

@andriinuts andriinuts commented Mar 10, 2024

Description

Fixes #2723
The reason was that we are calling applyVariantPrices even when the entity is null. And it happens only for ProductVariant.

Breaking changes

No

Checklist

📌 Always:

  • I have set a clear title
  • My PR is small and contains a single feature
  • I have checked my own PR

👍 Most of the time:

  • I have added or updated test cases
  • I have updated the README if needed

Copy link

netlify bot commented Mar 10, 2024

Deploy Preview for effervescent-donut-4977b2 canceled.

Name Link
🔨 Latest commit d257149
🔍 Latest deploy log https://app.netlify.com/sites/effervescent-donut-4977b2/deploys/65edee7d4cd60b0008a8c157

@andriinuts andriinuts force-pushed the fix/custom-field-product-variant branch from 3ed1f79 to d257149 Compare March 10, 2024 17:31
@michaelbromley michaelbromley merged commit b4f8a55 into vendure-ecommerce:master Mar 18, 2024
18 checks passed
@michaelbromley
Copy link
Member

Thank you!

@andriinuts andriinuts deleted the fix/custom-field-product-variant branch March 18, 2024 11:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants