-
-
Notifications
You must be signed in to change notification settings - Fork 202
Component: Support deep merging for bind_ configs #9280
Copy link
Copy link
Closed
Labels
aicoreCore framework functionalityCore framework functionalityenhancementNew feature or requestNew feature or request
Description
Currently, the bind_ configuration in Neo.component.Abstract lacks a deep merge descriptor. When a component instance is created and provided with a bind config (e.g., {store: 'stores.myStore'}), it completely overwrites any class-level bind config (e.g., {animateVisuals: ...}), rather than merging them.
This leads to unexpected loss of reactive bindings defined on the prototype.
Implementation Plan:
- Add
[isDescriptor]: trueandmerge: 'deep'to thebind_configuration insrc/component/Abstract.mjs. - Add unit tests in
test/playwright/unit/state/Provider.spec.mjsto explicitly verify that class-level and instance-level bindings are correctly deep-merged.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
aicoreCore framework functionalityCore framework functionalityenhancementNew feature or requestNew feature or request