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

Add Deep Linking (Plugin OAuth) #697

Closed
karaggeorge opened this issue Aug 7, 2019 · 5 comments · Fixed by #795
Closed

Add Deep Linking (Plugin OAuth) #697

karaggeorge opened this issue Aug 7, 2019 · 5 comments · Fixed by #795

Comments

@karaggeorge
Copy link
Member

Feature Request: deep linking

It would be nice to have deep linking for Kap, mainly for oauth purposes, or anything else we can think of. My main use-case would be for plugin oauth. Some services like Dropbox don't accept apps if the oauth happens in a "web view". They need it to happen in the user's browser. A way to get around that without having a server running, is to set the callback url to be a deep link to Kap, which can then receive the code. After that we'll need some kind of API for Kap to forward that code to the right plugin (and add that to the plugin docs).

If not able to deep link directly (some services require a https url), we might set up a dummy Now deployment that explains that it will open the app and deep link from there (similar to how Slack does it)

@karaggeorge
Copy link
Member Author

Another solution for #380

@karaggeorge karaggeorge self-assigned this Aug 7, 2019
@sindresorhus
Copy link
Member

We could use this API to register our URL scheme for deep-linking.

@MadeByDouglas
Copy link

yes, I would love to build a plugin for Kap for my how to video site https://relay.how and deep links would be good. Where do I go to get started to make a plugin? I didn't see info on the front page

@albinekb
Copy link
Collaborator

albinekb commented Jan 7, 2020

Here's the documentation for plugins: https://github.com/wulkano/kap/blob/master/docs/plugins.md @MadeByDouglas

We should probably add this to the readme?

@MadeByDouglas
Copy link

yeah that would be nice :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants