- Issue created by @hooroomoo
- 🇧🇪Belgium wim leers Ghent 🇧🇪🇪🇺
Based on 🐛 Prop form is not loading for components that haven't been published Active 's bug report (closed as a duplicate of this).
Let's start with a failing test. I'm assuming this is a back-end bug.
Therefore adding a code component test case to
\Drupal\Tests\experience_builder\Kernel\ComponentInputsFormTest
(introduced in e 🐛 SDCs with optional images without examples cannot be placed Active !) should be able to reproduce the failure. - 🇺🇸United States tedbow Ithaca, NY, USA
re #10
Let's start with a failing test. I'm assuming this is a back-end bug.
I don't think so. I think the root cause is the same as 🐛 Hovered preview for JS components in library not working Active .
basically before 📌 Add a route for PATCHing both a config entity and its auto-saved version together Active when clicked "add to components" it would send up the whole entity including compiled_js and slots and props. After #3519634 the client never actually makes a request to add the slots to the save config and probably the "layers" menu was always using the saved config and never using the auto-save. It just wasn't noticeable because you can't change slots after you add the component to the library.
- 🇺🇸United States tedbow Ithaca, NY, USA
You can test this by following the directions in the summary and then requesting
http://exp-d-core.test/xb/api/v0/config/js_component/component_name
This should have emptyslots
(andcompiled_js
)/xb/api/v0/config/auto-save/js_component/component_name
this should haveslots
filled
If you
git checkout e72f0d0b538ef6abefae89f0a8f14409dfd3ec03^
which is the commit before 📌 Add a route for PATCHing both a config entity and its auto-saved version together Activehttp://exp-d-core.test/xb/api/v0/config/js_component/component_name
should haveslots
filled (andcompiled_js
)/xb/api/v0/config/auto-save/js_component/component_name
this should haveslots
filled
The difference is the client now never actually sends a request to
/xb/api/v0/config/js_component/component_name
withslots
filled. - 🇺🇸United States tedbow Ithaca, NY, USA
see #3521819-12: Hovered preview for JS components in library not working anymore → for small MR that reverts a bit of 📌 Add a route for PATCHing both a config entity and its auto-saved version together Active that solve the problem on this issue
- 🇫🇮Finland lauriii Finland
Closing as a duplicate of 🐛 Hovered preview for JS components in library not working Active .