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.

Sign up for GitHub

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

Jump to bottom

chain computed lost reactivity when computed effect happens #11169

Closed
wanghanzhen opened this issue Jun 18, 2024 · 2 comments
Closed

chain computed lost reactivity when computed effect happens #11169

wanghanzhen opened this issue Jun 18, 2024 · 2 comments
Labels
🔨 p3-minor-bug Priority 3: this fixes a bug, but is an edge case that only affects very specific usage. scope: reactivity

Comments

@wanghanzhen
Copy link

Vue version

3.4.29

Link to minimal reproduction

https://play.vuejs.org/#eNqFU02P0zAQ/SsjX9KKKu1SLpR0JVgtAg6wgpW4WEKpMy0piW35I3QV5b/v2GnaSN2P23jmzZv3JpOWfdQ6bTyyFcusMKV2YNF5fc1lWWtlHLRgcDsDoWrtHRbQwdaoGhJqSrjkUihpHTSwDrhJ8gWrSiXTIb+h/NA6mUxhfQ0tlwBN2uSVR3izhgR+K1MVRAZE4byRQ5XL7kQkLok2PWr6YcAUlxhxxnBJzu7LGpV3r0rpZnC1WCxoPJfZvN8M7YQeDmtd5Q7pBZAVZRODY9i2JLTrsvllnjY3yg9RNh/xsRlzlqxsy126t0rSR4kKOQuuygrND+1KssrZqtceajkt/P+3mHPG42zIi78o/j2R39tDyHF2Z9CiaZCzU83lZoeuL9/++o4Hik/FWhW+IvQLxZ9oVeWDxh72ycuCZI9wUe3XeFql3N3b24NDaQdTQWhAdhHPGR3ZzQvWz3KX6TL20ZejLd6hofXRDmmD4zN+5nh1hN8oL93xjBf9weAh9m69FGH4Efg5F06Zh4nMa5z2YnoetdlT/1FdqK4iWcRFQ11khfHA8/FdUfH0BxAVPckOufnTEJzGk5ll+i59+551j5mtObM=

Steps to reproduce

Check the dom render results in 1 second timeout.

What is expected?

Hello World World World
Hello World World World

What is actually happening?

Hello World World World
Hello World

System Info

No response

Any additional comments?

No response
@wanghanzhen
Copy link
Author

It works fine with both v3.5.0-alpha.2 and v3.4.14

@wanghanzhen wanghanzhen mentioned this issue Jun 18, 2024
Closed
Doctor-wu added a commit to Doctor-wu/core that referenced this issue Jun 20, 2024
@Doctor-wu
fix(reactivity): fix side effect computed dirty level
452df9a 
close vuejs#11181, vuejs#11169
@Doctor-wu Doctor-wu mentioned this issue Jun 20, 2024
Merged
@Doctor-wu Doctor-wu added scope: reactivity 🔨 p3-minor-bug Priority 3: this fixes a bug, but is an edge case that only affects very specific usage. labels Jun 20, 2024
yyx990803 pushed a commit that referenced this issue Jun 22, 2024
@Doctor-wu
fix(reactivity): fix side effect computed dirty level (#11183)
3bd79e3 
close #11181, #11169
@yyx990803
Copy link
Member

closed via #11183

@github-actions github-actions bot locked and limited conversation to collaborators Jul 7, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
🔨 p3-minor-bug Priority 3: this fixes a bug, but is an edge case that only affects very specific usage. scope: reactivity
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix(reactivity): chained computeds lost reactivity wanghanzhen/core
3 participants
@yyx990803 @wanghanzhen @Doctor-wu