-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
registerUriHandler is not implemented #13169
Comments
@rschnekenbu this API is not marked as stubbed, that's why it does not appear on the comparator output. Can we put this on the backlog for API-compat? |
@tsmaeder, I will take over this task. Can you assign it to me please? Thanks! |
contributed on behalf of STMicroelectronics Signed-off-by: Remi Schnekenburger <rschnekenburger@eclipsesource.com>
fixes eclipse-theia#13169 contributed on behalf of STMicroelectronics Signed-off-by: Remi Schnekenburger <rschnekenburger@eclipsesource.com>
fixes eclipse-theia#13169 contributed on behalf of STMicroelectronics Signed-off-by: Remi Schnekenburger <rschnekenburger@eclipsesource.com>
fixes eclipse-theia#13169 contributed on behalf of STMicroelectronics Signed-off-by: Remi Schnekenburger <rschnekenburger@eclipsesource.com>
I tested out the pull request today and its it not working with our handler for auth0. Has this been tested/used yet?, thanks. Would love to get this working in our theia project if possible. We are using electron, not (browser) . Can someone show an example implementation. I am able to see calls into the new classes, but it still failing on the redirect |
Feature Description:
The function registerUriHandler is not implemented in theia.
It is defined in the VS code API.
This is needed to handle callbacks to theia through URIs, for example when creating an extension using GitHub OAuth.
The text was updated successfully, but these errors were encountered: