search

web-infra-dev / modern.js

Modern.js is a web engineering system, including a web framework and a npm package solution.
https://modernjs.dev/en/
MIT License
4.42k stars 367 forks source link

electron services不能如预期一样工作 #589

Closed ascodelife closed 2 years ago

ascodelife commented 2 years ago

版本信息

├── @modern-js/app-tools@1.3.1 -> .\node_modules\.pnpm\@modern-js+app-tools@1.3.1_esbuild@0.14.14+typescript@4.5.5\node_modules\@modern-js\app-tools
└─┬ @modern-js/plugin-electron@1.0.15 -> .\node_modules\.pnpm\@modern-js+plugin-electron@1.0.15_9d741c1e61e4000586c68f5e4ff4dde9\node_modules\@modern-js\plugin-electron
  └── @modern-js/app-tools@1.3.1 -> .\node_modules\.pnpm\@modern-js+app-tools@1.3.1_esbuild@0.14.14+typescript@4.5.5\node_modules\@modern-js\app-tools

问题详情

按照文档 https://modernjs.dev/docs/guides/features/electron/ipc/regist-services/main 去使用Modernjs封装的上层通信服务时，在渲染进程出现了报错。 image

由于Modernjs封装的上层通信服务是基于ipcmain和ipcrenderer封装的，因此我提供了可能与之相关的issue见 https://github.com/electron/electron/issues/7300 为了更进一步说明，附上一个便于复现的项目地址。

问题复现

https://github.com/ascodelife/electron-service-test

spcBackToLife commented 2 years ago

看你在代码中直接使用了 api: image

因此，参考： https://modernjs.dev/docs/guides/features/electron/develop#%E7%AA%97%E5%8F%A3%E4%B8%AD%E5%BC%80%E5%90%AF-node 这个文档。可以这样修改启动命令，或者在命令文档中这样： image

ascodelife commented 2 years ago

感谢解答，建议像monorepo那一章一样附上一个可复现的最小demo，另外感觉目前文档“开始”“指南”“API”三处相对比较独立，希望能重新组织一下最好能相互link起来

chenjiahan commented 2 years ago

感谢反馈，我们会持续优化文档的组织方式和关联内容~