-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
vsx-registry: fix the 'licenseUrl' link #8095
Conversation
- fixes the `licenseUrl` link for builtin and installed extensions. Signed-off-by: vince-fugnitto <vincent.fugnitto@ericsson.com>
Nice, I have tried it with the electron example, and I could open the license in my OS browser 👍 |
@kittaakos thank you for the review! Without the changes proposed in #8063, the API call to |
👍 Follow-up: #8097 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have verified it locally with the electron example; I could open the license for the built-ins in my OS browser. 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Tested on Ubuntu 18.04 with Chrome.
Thanks @vince-fugnitto
What it does
Fixes: #8094
The following pull-request fixes the
licenseUrl
link when attempting to open an extension's license from the vsx-editor.Previously, the url was incorrect as it would refer to the local
hosted
path. If the data is available, use it directly to open the link.How to test
builtin
,installed
andsearch result
extension:Review checklist
Reminder for reviewers
Signed-off-by: vince-fugnitto vincent.fugnitto@ericsson.com