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

Reactivity / store regression in 3.39.0 #6557

Closed
fbasso opened this issue Jul 22, 2021 · 1 comment · Fixed by #6559
Closed

Reactivity / store regression in 3.39.0 #6557

fbasso opened this issue Jul 22, 2021 · 1 comment · Fixed by #6559

Comments

@fbasso
Copy link

fbasso commented Jul 22, 2021

Describe the bug

Hi !

I noticed a regression from the 3.38.3 to the 3.39.0. When a store is defined inside an object, and given to a child component, this one is undefined if used with the $ sign in the script section.

See the simple REPL for a better explanation 😃

Thanks !

Reproduction

You can reproduce the issue in this REPL : https://svelte.dev/repl/dcceadde29ce439886ffff8260f331a6?version=3.39.0

As you can see on the display, Inner store is undefined. This is due to the line

$: console.log('Inner store', $store);

in Child.svelte.

Commenting this line, and the example works again. It works with the 3.38.3.

Logs

No response

System Info

System:
    OS: Windows 10 10.0.18363
    CPU: (8) x64 Intel(R) Core(TM) i7-8665U CPU @ 1.90GHz
    Memory: 7.72 GB / 15.82 GB
  Binaries:
    Node: 16.5.0 - C:\Program Files\nodejs\node.EXE
    Yarn: 1.22.5 - C:\Program Files (x86)\Yarn\bin\yarn.CMD
    npm: 7.20.0 - C:\Program Files\nodejs\npm.CMD
  Browsers:
    Chrome: 91.0.4472.164
    Edge: Spartan (44.18362.1593.0)
    Internet Explorer: 11.0.18362.1
  npmPackages:
    rollup: ^2.3.4 => 2.53.3
    svelte: 3.39.0 => 3.39.0

Severity

blocking an upgrade

@Conduitry
Copy link
Member

Fixed in 3.40.1 - https://svelte.dev/repl/dcceadde29ce439886ffff8260f331a6?version=3.40.1 - thanks for the report!

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

Successfully merging a pull request may close this issue.

2 participants