Skip to content

fix: resolve Map nested object modification#799

Merged
webfansplz merged 1 commit intovuejs:mainfrom
rbgmulmb:fix/StateEditor-nested-value
Feb 26, 2025
Merged

fix: resolve Map nested object modification#799
webfansplz merged 1 commit intovuejs:mainfrom
rbgmulmb:fix/StateEditor-nested-value

Conversation

@rbgmulmb
Copy link
Contributor

Currently, modifying a nested object in a Map in Pinia causes an error.
image
image

This PR contains a fix for that issue and a new test.

fix(state): improve handling of Set in StateEditor
@netlify
Copy link

netlify bot commented Feb 13, 2025

Deploy Preview for vue-devtools-docs canceled.

Name Link
🔨 Latest commit 38a57f4
🔍 Latest deploy log https://app.netlify.com/sites/vue-devtools-docs/deploys/67ae0dc21009960008325103

Copy link
Collaborator

@Azurewarth0920 Azurewarth0920 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@webfansplz webfansplz changed the title Fix modifying a nested object in a Map fix: resolve Map nested object modification Feb 26, 2025
@webfansplz webfansplz changed the title fix: resolve Map nested object modification fix: resolve Map nested object modification Feb 26, 2025
@pkg-pr-new
Copy link

pkg-pr-new bot commented Feb 26, 2025

Open in Stackblitz

@vue/devtools-applet

npm i https://pkg.pr.new/@vue/devtools-applet@799

@vue/devtools-core

npm i https://pkg.pr.new/@vue/devtools-core@799

@vue/devtools-api

npm i https://pkg.pr.new/@vue/devtools-api@799

@vue/devtools

npm i https://pkg.pr.new/@vue/devtools@799

@vue/devtools-kit

npm i https://pkg.pr.new/@vue/devtools-kit@799

vite-plugin-vue-devtools

npm i https://pkg.pr.new/vite-plugin-vue-devtools@799

commit: 38a57f4

@webfansplz webfansplz merged commit 60fb42d into vuejs:main Feb 26, 2025
8 checks passed
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 this pull request may close these issues.

3 participants