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

#7981 has a bug with init binding #8146

Closed
shirotech opened this issue Dec 25, 2022 · 2 comments
Closed

#7981 has a bug with init binding #8146

shirotech opened this issue Dec 25, 2022 · 2 comments

Comments

@shirotech
Copy link

shirotech commented Dec 25, 2022

Describe the bug

Regarding #7981

When binding outside component, it has a breaking change and it has affected our production application massively, is it possible to restore the previous behavior?

Edit: Actually I see that #8114 is in the works, and hopefully can resolve the issue.

Reproduction

Breaking version:
https://svelte.dev/repl/09bb24de9beb4fdc9dd5bceaf6d6e1b0?version=3.54.0

Last known working version:
https://svelte.dev/repl/09bb24de9beb4fdc9dd5bceaf6d6e1b0?version=3.53.1

Also in latest version:
https://svelte.dev/repl/09bb24de9beb4fdc9dd5bceaf6d6e1b0?version=3.55.0

Logs

Breaking change in version: 3.54.0

From comp: 8
Outside comp: 0

Last known working version: 3.53.1

From comp: 8
Outside comp: 8

The previous and expected behavior is that the two values both from parent and child should be the same.



### System Info

```shell
System:
    OS: macOS 13.0.1
    CPU: (16) x64 Intel(R) Core(TM) i9-9880H CPU @ 2.30GHz
    Memory: 32.32 GB / 64.00 GB
    Shell: 5.8.1 - /bin/zsh
  Binaries:
    Node: 16.19.0 - ~/.nvm/versions/node/v16.19.0/bin/node
    Yarn: 1.22.19 - /usr/local/bin/yarn
    npm: 8.19.3 - ~/.nvm/versions/node/v16.19.0/bin/npm
  Browsers:
    Chrome: 108.0.5359.124
    Firefox: 107.0.1
    Safari: 16.1


### Severity

blocking an upgrade
@lukaszpolowczyk
Copy link

Oh, there it is already.

I described with an example of the function:
#8152

@baseballyama
Copy link
Member

Close this now because #8114 is merged.

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

3 participants