reflex-app / reflex-app/reflex
Move Electron Remote calls to ipcRender.invoke()
Open
Nobody has claimed this yet.
Type: Enhancement
Type: Performance
- Dominant language
- Vue
- Stars
- 34
- Forks
- 1
- PR merge metrics
- No merged PRs in 30d
Description
(Depends on Electron 7.0)
It is suggested to avoid using the remote module of Electron for security and performance issues. Electron 7.0 has some new functions to use instead.
- https://electronjs.org/blog/electron-7-0
- https://medium.com/@nornagon/electrons-remote-module-considered-harmful-70d69500f31
- Replace all
remotecalls with@electron/remote - Standardize a way to organize communications between main and renderer process (IPC)
- Refactor requests like
remote.app.nameandremote.app.versionso that this data is available when the app loads. Maybe store this info in Vuex Store?
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by locating every use of Electron's remote module, then read the linked Electron 7.0 migration guidance and review how main and renderer processes currently communicate. Done means remote calls are replaced, IPC communication is organized consistently, and app name and version are available when the app loads.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- electron
- Domain
- desktop
- Issue type
- Refactor
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 25/100