Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Potential breaking change in upcoming Svelte version #5

Open
rmunn opened this issue Sep 27, 2021 · 0 comments
Open

Potential breaking change in upcoming Svelte version #5

rmunn opened this issue Sep 27, 2021 · 0 comments

Comments

@rmunn
Copy link

rmunn commented Sep 27, 2021

In sveltejs/svelte#6750, there is a proposed change to the StartStopNotifier type of Svelte stores, which will be a breaking change for anyone creating custom stores. See the PR for details, but the short version is that if that PR is accepted and merged into Svelte, you'll need to make some small changes to your custom store to maintain Svelte compatibility, as follows:

  1. Inside your writable function, change stop = start(set) || noop; to stop = start(set, update) || noop; in the subscribe() implementation.
  2. Inside your derived function, change return readable(initial_value, (set) => { to return readable(initial_value, (set, update) => { and then change const result = fn(single ? values[0] : values, set); to const result = fn(single ? values[0] : values, set, update);

In your repo, the affected code appears to be in store.js.

If you have any comments about the proposed breaking change, please come leave a comment on the Svelte PR to let me know whether it would make your life easier or harder.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant