- Issue created by @tedbow
- Merge request !1030Draft: #3524274 Use auto-save code components in list โ (Open) created by tedbow
- ๐บ๐ธUnited States tedbow Ithaca, NY, USA
This is rough POC but I think it solves the problem in ๐ JS component slots don't appear in the preview canvas until published Active while allowing the client to only send
status: true
when adding the code component to the library.the other thing this would allow is adding new props and slots after a component has been added to the library. there are probably other hurdles to overcome for this but this does allow the component list use the auto-save version of code components which would not affect currently published pages with published code components in them
- ๐บ๐ธUnited States hooroomoo
Adding related issue. Added todos in there to point to this one.
- ๐บ๐ธUnited States hooroomoo
Marking as stable blocker to remove the band-aid that currently covers up a bug.
- ๐ง๐ชBelgium wim leers Ghent ๐ง๐ช๐ช๐บ
๐ค Iโm not sure I follow whatโs going on here, what exactly is proposed, and why the draft MRโs direction is the right one.
AFAICT the draft MR is making the API returning the list of config entities of some type to no longer return whatโs actually stored, but instead making it return the auto-saved (not really saved/live) config entity.
That means it becomes impossible to retrieve the non-auto-saved-but-actual config entity.
If I ignore that and try to follow this MRโs rationale then it seems like this argues ๐ Add a route for PATCHing both a config entity and its auto-saved version together Active was wrong, because when sending
PATCH status: true
, youโd want to use the auto-save entry, not the live data?
But if so, why is this MR only modifying
::list()
, not::get()
and::patch()
? - ๐ง๐ชBelgium wim leers Ghent ๐ง๐ช๐ช๐บ
Given the above, I donโt understand how this can be a bug report. The โlist config entities of typeโ internal HTTP API works as intended? ๐ซฃ
Perhaps an issue update will illuminate me ๐
- ๐บ๐ธUnited States tedbow Ithaca, NY, USA
I will update the summary.