- Issue created by @hooroomoo
- πΊπΈUnited States phenaproxima Massachusetts
hooroomoo β credited phenaproxima β .
- πΊπΈUnited States tedbow Ithaca, NY, USA
hooroomoo β credited tedbow β .
- π§πͺBelgium wim leers Ghent π§πͺπͺπΊ
hooroomoo β credited wim leers β .
- πΊπΈUnited States hooroomoo
transferring credits from β¨ Content templates, the boss battle: create a UI for editing templates Active
- πΊπΈUnited States hooroomoo
I am not sure what the difference between the backend items in the map:
HTTP endpoint: dynamic props sourcesandHandle dynamic propsis...I wonder if its the same thing but we just didn't realize when we created this map together. I see only 4 backend tickets got created instead of 5 so maybe it is a duplicate? But I'm not certain.
- π³π±Netherlands balintbrews Amsterdam, NL
hooroomoo β credited balintbrews β .
- πΊπΈUnited States tedbow Ithaca, NY, USA
This how issue is about enabling the UI. There has already been a lot of work to make the content template on the back-end. Although there are backend issues they are in support of the UI
- πΊπΈUnited States tedbow Ithaca, NY, USA
Wanted to make a clarification here based on call between myself, @balintbrews, @hooroomoo and fmazeikis.
Current ContentTemplates require at least 1 dynamic prop. This makes sense but this requirement would force use to do issues in certain order and require larger issues. Since all of this work is on 1.x which does not have a release we deemed to ok if in the first few issues Content Templates don't actually require dynamic props and don't have a way to set them.
The requirement will be removed in π Refactor ApiLayoutController into 2 sub-class to support Content Templates Active which is the first major back-end issue. It will be added back near the end in π Require content templates to have at least one dynamic property source Active the actually requirement is 1 schema file change a little test coverage. Details in the related issues
- π§πͺBelgium wim leers Ghent π§πͺπͺπΊ
π Move `PropSourceEndpointTest` into new `XbConfigEntityHttpApiTest::testComponent()` Active is in π₯³ The test coverage for that revealed one bug in shape matching, which isn't a hard blocker, but needs to be fixed at some point. So created a new section.
β¨ Move entity type and ID from base path and into routing parameters Active is unblocked because β¨ Allow working with a new (unsaved) entity Active landed! (Which means that one was missing, so added it π)
- π§πͺBelgium wim leers Ghent π§πͺπͺπΊ
I doubt that we'd be okay with only supporting populating SDC-sourced component instances with structured data β we of course also want to be able to do that for code components. Which means that we have to do #3503038. See #3503038-16: Enable candidate `DynamicPropSource` suggestions for code components: refactor `GeneratedFieldExplicitInputUxComponentSourceBase` and `FieldForComponentSuggester` to need only SDC's ComponentMetadata, not SDC plugin instances β for details.
- π§πͺBelgium wim leers Ghent π§πͺπͺπΊ
We've got kind of a meta sprawl going on here π π i.e.: what is the difference in intent/scope between:
- this issue
- vs π± [META] 7. Content type templates β aka "default layouts" β clarify the tree+props data model Active
- vs π [PoC] Introduce a `ContentTypeTemplate` config entity Active
So together with @f.mazeikis and @thoward216, clarifying the scope of this meta! π
- Per @lauriii months ago, we won't be building exposed slots until after 1.0.
- Per @lauriii and @effulgentsia, the
ContentTemplates scope for 1.0 is limited to onlyNodeentities β since https://git.drupalcode.org/project/experience_builder/-/commit/843e7b0b1.... - no XB-native view mode creation UI
- πΊπΈUnited States hooroomoo
Moving the
Allow linking a component prop of a template to a dynamic fieldto be higher in the FE list for visibility and higher priority because this could be big one due to it also has to make sure that the component props form can be correctly generated after a dynamic field is set to a prop.https://git.drupalcode.org/project/experience_builder/-/merge_requests/1...
- πΊπΈUnited States tedbow Ithaca, NY, USA
Added π ContentTemplate status should be used to determine if it should be used. Active to the list
- πΊπΈUnited States effulgentsia
Per π± Milestone 1.0.0: Production Sites Postponed this is a requirement for an XB 1.0.0 stable release, so tagging it as such.
- πΊπΈUnited States tedbow Ithaca, NY, USA
- π§πͺBelgium wim leers Ghent π§πͺπͺπΊ
π ContentTemplate status should be used to determine if it should be used. Active landed.
- πΊπΈUnited States hooroomoo
Confirmed with product these are out of scope for 1.0 so updating the issue summary
Assigned to: lauriii β¨ Render template data form Active
β¨ Allow creating/deleting view mode from UI Active - π§πͺBelgium wim leers Ghent π§πͺπͺπΊ
π Refactor ApiLayoutController into 2 sub-class to support Content Templates Active is in.
- π§πͺBelgium wim leers Ghent π§πͺπͺπΊ
π Create HTTP Endpoint for Content Template Preview Entities Active is in.
- πͺπΈSpain penyaskito Seville π, Spain πͺπΈ, UTC+2 πͺπΊ
Added π `CanvasUiAccessCheck` should grant access if the user has access to content templates or code components Active to the IS.
- π§πͺBelgium wim leers Ghent π§πͺπͺπΊ
π Create HTTP Endpoint for View modes/content templates Active is in.
- π§πͺBelgium wim leers Ghent π§πͺπͺπΊ
And π Create HTTP Endpoint for Content Template Preview Entities Active no longer needs a follow-up! π₯³ Yay more green.
- πΊπΈUnited States effulgentsia
Not every stable blocker needs to block an RC, but this one does.
- π§πͺBelgium wim leers Ghent π§πͺπͺπΊ