Closed (fixed)
Project:
Experience Builder
Version:
0.x-dev
Component:
Page builder
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
13 Feb 2025 at 11:50 UTC
Updated:
18 Mar 2025 at 10:44 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #3
balintbrewsComment #4
nagwani commentedComment #6
balintbrewsComment #7
balintbrewsComment #9
balintbrewsCrediting @effulgentsia for suggesting to use default config for the asset library storing the global CSS.
Comment #10
larowlanWhen I try to publish this, I get an error from the publish endpoint
'This value should not be blank'
Here's the detail
Will try to fix it
Comment #11
larowlanThe name is indeed blank
Comment #12
larowlanCreating a new component
1) Name is sent
2) Create another component
2) GET request starts but is aborted
3) PATCH request runs but sends the value of name
4) Name is now set to old value
I think the same thing is occurring with a clean state - but instead of sending the old component name, it is sending an empty value.
I think we can fix this in the 'create component' dialog, going to try that
Comment #13
larowlanThe issue isn't coming from the add dialog, its from the code added here.
Because we save an autosave entry before we get data from the backend, we are relying on the name from code editor slice.
This is empty if the component is new and the first component created here.
It is the name of the previously edited component if not the first one.
I think this highlights the brittleness of the 204 from the backend and we should reconsider a 307 so that the fetch always gets some data.
Comment #14
larowlanActually, ignore #13 - I was able to resolve this from the add dialog.
I'll add a test.
Comment #15
larowlanFixed #11 and #12 and added a test
There's a couple of open comments on the MR but other than that I think this is RTBC
Comment #18
balintbrewsComment #19
balintbrewsComment #20
nagwani commented