Skip to content
New issue

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

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

Already on GitHub? # to your account

fix(client): prevent use side effect ref in computed #1501

Closed
wants to merge 1 commit into from

Conversation

Doctor-wu
Copy link

close #1500

Before submitting the PR, please make sure you do the following

  • Read the Contributing Guidelines.
  • Provide a description in this PR that addresses what the PR is solving. If this PR is going to solve an existing issue, please reference the issue (e.g. close #123).

What is the purpose of this pull request?

  • Bug fix
  • New feature
  • Documentation update
  • Other

Description

prevent use side effect ref in computed
close #1500

Screenshots

Before

After

@meteorlxy
Copy link
Member

b352c09

@meteorlxy meteorlxy closed this Feb 19, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug report] getters-should-be-side-effect-free
2 participants