search

imodeljs / desktop-starter

An iTwin.js Desktop starter app
https://itwinjs.org
MIT License
21 stars 12 forks source link

"imodeljs/desktop-starter" failed to initialize electron IPC #32

Closed namsancow closed 4 years ago

namsancow commented 4 years ago

Yesterday, "imodeljs/desktop-starter" was updated as 2.6.0. And I tested the new version ""imodeljs/desktop-starter" on Win 10 64bit & Chrome.

The process of "npm install" is ok! The process of "npm build" is also ok! However, after messaging of "compiled successfully", there is an unseen warning in the previous version as following:

image

And after loading "desktop-starter" on electron, there is also an unseen warning in the previous version as following:

image image image

And the default model is not displayed. This issue is not seen in previous versions. If I don't understand it well, please let me know.

calebmshafer commented 4 years ago

@namsancow I'm not able to reproduce the Electron IPC failure but did hit an issue with the use of getRemote() on the frontend. This failure is due to the removal of nodeIntegration for Electron in iModel.js 2.6.0. I pushed a branch with that temporarily removed. Can you checkout the branch fix-arg-parsing and run the following to see if it works for you?

  1. npm i
  2. npm run build
  3. npm run electron

I'm working on a different way to handle arg parsing now.