Skip to content

fix: resolve Map nested object modification - #799

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

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

Conversation

@rbgmulmb

Copy link
Copy Markdown
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

netlify Bot commented Feb 13, 2025

Copy link
Copy Markdown

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

@Azurewarth0920 Azurewarth0920 left a comment

Copy link
Copy Markdown
Collaborator

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

pkg-pr-new Bot commented Feb 26, 2025

Copy link
Copy Markdown

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
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