search

w11k / Tydux

Type safe state management for web applications
Apache License 2.0
36 stars 4 forks source link

Use DeepPartial for state consumer #28

Open romanroe opened 4 years ago

sengmann commented 1 year ago

@romanroe Do we want to change our implementation? Otherwise I would like to close this issue.

romanroe commented 1 year ago

@sengmann Currently the Facade#select() returns Observable<Readonly<S>>, which only marks the state's props as readonly (shallow). We should use DeepPartial since the whole state is frozen, not only direct properties.