- 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_nameThis should have emptyslots(andcompiled_js)/xb/api/v0/config/auto-save/js_component/component_namethis should haveslotsfilled
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_nameshould haveslotsfilled (andcompiled_js)/xb/api/v0/config/auto-save/js_component/component_namethis should haveslotsfilled
The difference is the client now never actually sends a request to
/xb/api/v0/config/js_component/component_namewithslotsfilled. - 🇺🇸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 .