Implement onstate and onupdate hooks #1344
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Still to do:
add deprecation warnings when, and addthis.observe
is calledthis.observe
to svelte-extras (which can't happen until this is released, as it will depend on thestate
andupdate
events).An unanticipated effect, which I think is actually a positive, is that we need
store.on
andstore.fire
. So apps now get a global event bus for free.(store.onchange
also needs to be deprecated.)It might also be possible to replace
_bind
withcomponent.on('state')
, which would be nice.