Skip to content
This repository was archived by the owner on Jun 17, 2020. It is now read-only.

Added support for promises in executeMainProcessMethod - #30

Open
jcabot21 wants to merge 1 commit into
electron-userland:masterfrom
jcabot21:async-mainProcessMethod
Open

jcabot21 wants to merge 1 commit into
electron-userland:masterfrom
jcabot21:async-mainProcessMethod

Conversation

@jcabot21

Copy link
Copy Markdown

This fixes #29 by behaving like evalRemoteMethod.

@queesamor

Copy link
Copy Markdown

I implemented this patch manually in my app's code since I was running into the exact issue described in #29, rendering my code useless, and these proposed changes resolved the issue. I have not found any regressions so far in usage.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Returning a Promise from 'executeMainProcessMethod'

3 participants