Tydux icon indicating copy to clipboard operation
Tydux copied to clipboard

Published 20 hours ago • verified publisher icon w11k

Use DeepPartial for state consumer

Open romanroe opened this issue 4 years ago • 2 comments

romanroe avatar Feb 29 '20 09:02 romanroe

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

sengmann avatar Dec 14 '22 14:12 sengmann

@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.

romanroe avatar Dec 22 '22 10:12 romanroe