-
Notifications
You must be signed in to change notification settings - Fork 60
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
Added support to create variants on object creation #375
Conversation
d594549
to
91b9ff6
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Small improvements
2ac1474
to
5a33deb
Compare
can you please just add the english translation. all others are maintained via poeditor. thx very much. |
Co-authored-by: Sebastian Blank <[email protected]>
As the parent can be changed for variants this is now catched, as a variant cannot change its parent anymore.
CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅ |
Qodana for PHPIt seems all right 👌 No new problems were found according to the checks applied 💡 Qodana analysis was run in the pull request mode: only the changed files were checked Contact Qodana teamContact us at [email protected]
|
As the parent can be changed for variants this is now catched, as a variant cannot change its parent anymore.
7ec2d92
to
91db3b3
Compare
b55dcf2
to
a5836ea
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would do one minor adaption, besides that it looks good to me 👍
…importer into feature/support-variants
2227ac3
to
c457958
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
finally it looks good. thank you so much for your support! this is a great addition.
Allows the import of variants on
Element Creation
andElement Location Update
, when using thefindParent
location strategy.The type of the element will then be set to variant if all of the following conditions apply:
As Variant
checkbox is checkedAdditionally two more variant related checks were added:
variant