Skip to content

feat: add protocol api#236

Draft
lellimecnar wants to merge 1 commit intoMelvin-Abraham:masterfrom
lellimecnar:api
Draft

feat: add protocol api#236
lellimecnar wants to merge 1 commit intoMelvin-Abraham:masterfrom
lellimecnar:api

Conversation

@lellimecnar
Copy link
Copy Markdown

@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
Copy Markdown
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
Copy Markdown
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