Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: add protocol api #236

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from
Draft

Conversation

lellimecnar
Copy link

@lellimecnar lellimecnar commented Mar 11, 2021

DO NOT MERGE!!!

I was just playing around with using a gassist protocol to open up the app for an API.

DO NOT MERGE!!!

@lellimecnar
Copy link
Author

I was able to get it working enough to set up an Alfred workflow that uses the app to trigger queries, but it doesn't show the window if it's not already visible. I'm not very familiar with Electron, but I thought I'd take a crack at it.

@Melvin-Abraham
Copy link
Owner

That seems like a great start. This could maybe also used for CLI implementation going forward.

Note: The logic for custom protocol can be moved to the main process (app.js as per current setup) instead of the renderer process (main.js as per current setup) and when the request is successfully captured by the main process, it could be relayed to the renderer process via IPC.

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