Skip to content

Add vue devtools. - #34

Open
frankperez87 wants to merge 1 commit into
Deluze:masterfrom
frankperez87:feature/add-vue-devtools
Open

Add vue devtools.#34
frankperez87 wants to merge 1 commit into
Deluze:masterfrom
frankperez87:feature/add-vue-devtools

Conversation

@frankperez87

Copy link
Copy Markdown

This adds Vue Devtools support.

One thing to note is that you will get a warning stating:
Manifest version 2 is deprecated, and support will be removed in 2023.

This is already being addressed within the vuejs devtools repo here
https://github.com/vuejs/devtools/issues/1917

@frankperez87 frankperez87 mentioned this pull request Nov 24, 2022
@Deluze

Deluze commented Nov 24, 2022

Copy link
Copy Markdown
Owner

Thanks for this PR!

I would like to merge this, but I'm running against some errors locally:

[electron] [38620:1124/235749.957:ERROR:CONSOLE(160)] "Electron sandbox_bundle.js script failed to run", source: node:electron/js2c/sandbox_bundle (160)
[38620:1124/235749.957:ERROR:CONSOLE(160)] "TypeError: object null is not iterable (cannot read property Symbol(Symbol.iterator))", source: node:electron/js2c/sandbox_bundle (160)

I'm not sure what causes this yet, but doing a quick google search it seems more people have ran into this issue. This error pops up on start, and more of them get logged after opening the Vue devtools.

Another issue is that the Content Security Policy header is blocking a request to a script that Vue Devtool is trying to load and execute... thinking of omitting the header and let the developers decide what they want to do with it.

[electron] [51244:1124/235652.437:ERROR:CONSOLE(0)] "Refused to load the script 'https://cdn.headwayapp.co/widget.js' because it violates the following Content Security Policy directive: "script-src 'self'". Note that 'script-src-elem' was not explicitly set, so 'script-src' is used as a fallback.

@Deluze

Deluze commented Nov 24, 2022

Copy link
Copy Markdown
Owner

I'm thinking that the remote vue devtools package might be a better (temporary) solution until these errors are fixed. Only downside is that the devtools is a separate process on the machine, at least the developer can then decide if they want to launch the devtools or not.

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.

2 participants