-
-
Notifications
You must be signed in to change notification settings - Fork 45
Breeze integration #190
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
base: main
Are you sure you want to change the base?
Breeze integration #190
Conversation
|
Amazing stuff @vovayatsyuk, very grateful for this MR 😍 I've read your comment on the discussion I've started and it seems you've overcome the questions. I am working on some improvements and I wonder if we can make this work for versions before 2.26.1. So I'll leave this PR open until that stuff gets merged in and will take a look into this also installing the theme. In the meanwhile, could you maybe add a little component to a random page and make a quick screencast from it? Would already love to see something on screen. Once again, great contribution and thanks for putting in the time! |
This commit restores class names on the components' root element before morph.
|
Made a video of minimal component on the product page: View videoScreencast.from.08.10.25.16.58.24.webm |
|
Just stumbled with weird behavior. I see the 404 request to It's coming from alpinejs.phtml |
|
I think for now, this is save to merge right? Some upgrades for the next beta will probably fix more stuff in terms of the 404 you've noticed. Can you confirm this can be merged? |
|
Wanted to check if everything is ok, but "Increment" button doesn't work anymore. Request to
|
|
Try and see what happens when using |
|
Still the same It's caused by exception thrown by I've debugged further to see what caused the issue and found this Exception throwm by Magento: https://github.com/magento/magento2/blob/2.4.8-p3/lib/internal/Magento/Framework/Reflection/TypeProcessor.php#L924-L934 I've tried if DocBlock will help:
p.s. I'm using Magento 2.4.8-p3 |
Note: this integration requires the 2.26.1 version of Breeze module because of new
$.breeze.getFormKey()method.