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

[vscode][plugin] Respect when condition for vscode plugins #6015

Closed
vinokurig opened this issue Aug 22, 2019 · 5 comments · Fixed by #6068
Closed

[vscode][plugin] Respect when condition for vscode plugins #6015

vinokurig opened this issue Aug 22, 2019 · 5 comments · Fixed by #6068
Labels
bug bugs found in the application help wanted issues meant to be picked up, require help plug-in system issues related to the plug-in system vscode issues related to VSCode compatibility
Milestone

Comments

@vinokurig
Copy link
Contributor

vinokurig commented Aug 22, 2019

Description

Some vscode extensions have when condition provided in hair pacage.json file, e.g. the vscode github-pull-request extension has such condition in the view contribution: https://github.com/microsoft/vscode-pull-request-github/blob/d91e778705cd5af0785c5a05184ebc550334b409/package.json#L182
Since #5855 was applied the plugin view is unavailable.

Reproduction Steps

  1. Remove git from theia/examples/browser/package.json.
  2. Download vscode git extension.
  3. Create plugins folder in Theia's root and paste the plugin to it.
  4. Clone the vscode github extension and compile it.
  5. Start the github plugin as a Hosted Plugin.
  6. Click the GitHub icon.

WRONG: The view is unavailable:
screenshot-che-che 192 168 39 215 nip io-2019 08 15-10-21-47

OS and Theia version:

Diagnostics:

@vinokurig vinokurig added plug-in system issues related to the plug-in system todo larger scoped issues vscode issues related to VSCode compatibility labels Aug 22, 2019
@akosyakov akosyakov removed the todo larger scoped issues label Aug 23, 2019
@akosyakov akosyakov added the bug bugs found in the application label Aug 23, 2019
@vinokurig
Copy link
Contributor Author

@akosyakov

@vinokurig I believe it is already supported: https://github.com/theia-ide/theia/blob/5339bef84b447d457b15432f2b6a08368f95765f/packages/plugin-ext/src/main/browser/view/plugin-view-registry.ts#L112-L121

It is, but it doesn't respect condition from the GitHub plugin: "when": "!config.githubPullRequests.showInSCM && config.git.enabled"
https://github.com/microsoft/vscode-pull-request-github/blob/d91e778705cd5af0785c5a05184ebc550334b409/package.json#L182
I'll rename the issue title

@vinokurig vinokurig changed the title [vscode][plugin] Support when condition for vscode plugins [vscode][plugin] Respect when condition for vscode plugins Aug 23, 2019
@akosyakov
Copy link
Member

All preferences contributed by VS Code extensions should be already supported in when closure. These preferences are contributed by git and git pull request extensions, not by Theia. Are you sure that you have VS Code git extension installed?

@vinokurig
Copy link
Contributor Author

@akosyakov

Are you sure that you have VS Code git extension installed?

yes, I can see contributed git preferences:
screenshot-localhost-3030-2019 08 23-14-15-13
But they are not registered in the ContextKeyService

@akosyakov
Copy link
Member

@akosyakov akosyakov added the help wanted issues meant to be picked up, require help label Aug 25, 2019
@akosyakov akosyakov added this to the 0.10.0 milestone Aug 27, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug bugs found in the application help wanted issues meant to be picked up, require help plug-in system issues related to the plug-in system vscode issues related to VSCode compatibility
Projects
None yet
2 participants