Problem/Motivation

Evaluate and adapt instance list. We should be closer to EntityListBuilder which is not possible currently as we use State storage.

Currently based on a custom method in an helper to collect config. Each instance if not in state is then instantiated to allow it existence in the list.

Advantage : The list is sync with config, allow deletion of all states, and when click "build display" the instance is really created from config.

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

Comments

mogtofu33 created an issue. See original summary.

mogtofu33’s picture

mogtofu33 commented
Title: Instance list » Instance list refactor

pdureau’s picture

pdureau commented
Issue tags: -display_builder-1.0.0-beta2 +display_builder-1.0.0-beta4
pdureau’s picture

pdureau commented
Assigned: Unassigned » mogtofu33

Currently based on a custom method in an helper to collect config. Each instance if not in state is then instantiated to allow it existence in the list.

For your information, #3549266: Move DisplayBuildableInterface to a new plugin type is replacing the custom method in an helper to collect config (DisplayBuilderUiHelpers) by DisplayBuildableInterface::collectInstances()

What is left to do here?

mogtofu33’s picture

mogtofu33 commented
Assigned: mogtofu33 » Unassigned
Status: Active » Fixed

Right, work is done in #3549266: Move DisplayBuildableInterface to a new plugin type, we can close.

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

pdureau’s picture

pdureau commented
Issue tags: -display_builder-1.0.0-beta4 +display_builder-1.0.0-beta3

cool :)

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.