Skip to content

fix(vite): use embedded mode on vite-plugin-inspect page - #753

Open
LukerSpringtree wants to merge 1 commit into
vuejs:mainfrom
LukerSpringtree:fix/736
Open

LukerSpringtree wants to merge 1 commit into
vuejs:mainfrom
LukerSpringtree:fix/736

Conversation

@LukerSpringtree

Copy link
Copy Markdown
Contributor

fix to #736 by update "vite-plugin-inspect";
add embedded option to hide some buttons.

@netlify

netlify Bot commented Dec 22, 2024 •

Copy link
Copy Markdown

✅ Deploy Preview for vue-devtools-docs canceled.

Name Link
🔨 Latest commit fb77993
🔍 Latest deploy log https://app.netlify.com/sites/vue-devtools-docs/deploys/67676a4a544a5c0008f3eae4

@LukerSpringtree

Copy link
Copy Markdown
Contributor Author

I removed some dependencies in the playground folder. Because it's embedded in the tool they don't need them.

@alexzhang1030 alexzhang1030 changed the title fix: fix to #736 by update "vite-plugin-inspect" fix(vite): use embedded mode on vite-plugin-inspect page Dec 23, 2024
@alexzhang1030

Copy link
Copy Markdown
Member

Hey, your lockfile is not up-to-date.

@LukerSpringtree

LukerSpringtree commented Dec 23, 2024 •

Copy link
Copy Markdown
Contributor Author

Hey, your lockfile is not up-to-date.

@alexzhang1030
image

Maybe is the network. Mabe is another reason. I can't update the lockfile correctly! Can you help me, please!

@LukerSpringtree

LukerSpringtree commented Dec 23, 2024 •

Copy link
Copy Markdown
Contributor Author

Hey, your lockfile is not up-to-date.
@alexzhang1030
Bro, may I ask if there’s an issue with our contribute file?
Every time I fork the devtool repository and run pn i, it takes a long time and has no results.
I have to run pn i -w -f first, and then run pn i again (can't stop auto) to get the project up and running.
It’s so frustrating. Why is this happening? I appreciate any help you can provide.

@webfansplz

webfansplz commented Dec 23, 2024 •

Copy link
Copy Markdown
Member

In order to support vite 5 and 6, we should pin vite-plugin-inspect to the v0.8.x. So let's hold on to this feature for now.

See:#713

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants