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

fix(reactivity): Pass oldValue to triggerRefValue to satisfy interface DebuggerEventExtraInfo #8210

Open

Azurewarth0920 wants to merge 2 commits into vuejs:main from Azurewarth0920:fix/ref-effective-interface

Azurewarth0920 commented May 2, 2023 •

edited

fix: vuejs/pinia#2061

The DebuggerEventExtraInfo triggered by a computed ref doesn't have the field oldValue.

For consistency (consistent with reactive), added oldValue to debuggerEventExtraInfo.

Azurewarth0920 added 2 commits

May 2, 2023 12:02


          fix: Add old value to triggerEffects' dependency

e065e6e


          chore: Add test

428c680

sxzz force-pushed the main branch from 39d4fc1 to 1b56405 Compare

August 21, 2023 05:00

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment