Skip to content

Fix #9761: Hosted plugin can't be stop or restart#9780

Merged
colin-grant-work merged 1 commit intoeclipse-theia:masterfrom
EstherPerelman:fix-9761
Jul 27, 2021
Merged

Fix #9761: Hosted plugin can't be stop or restart#9780
colin-grant-work merged 1 commit intoeclipse-theia:masterfrom
EstherPerelman:fix-9761

Conversation

@EstherPerelman
Copy link
Contributor

@EstherPerelman EstherPerelman commented Jul 27, 2021

Signed-off-by: Esther Perelman esther.perelman@sap.com

What it does

Fix #9761 by removing a problematic import...

How to test

  1. Unzip test-extension.zip
  2. Run inside: yarn & yarn run compile
  3. Open Theia
  4. Run ctrl+shift+p -> Hosted Plugin: Start Instance
  5. Select the downloaded extension
  6. Stop or Restart (by pressing on the left bottom bar or ctrl+shift+p -> Hosted Plugin: Stop/Restart Instance
  7. See no error

Review checklist

Reminder for reviewers

Signed-off-by: Esther Perelman <esther.perelman@sap.com>
@EstherPerelman
Copy link
Contributor Author

@vince-fugnitto @colin-grant-work Can you approve & merge this small but critical fix? for the next version 1.16?

@colin-grant-work
Copy link
Contributor

@EstherPerelman, I'd be happy to take a look. Would it be possible for you link to the source for the plugin to test? In my test environment, I can't download .zip files.

Copy link
Member

@vince-fugnitto vince-fugnitto left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I confirmed that the change allows us to stop and restart the hosted-plugin using the example provided.

@vince-fugnitto vince-fugnitto added the plug-in system issues related to the plug-in system label Jul 27, 2021
@EstherPerelman
Copy link
Contributor Author

@EstherPerelman, I'd be happy to take a look. Would it be possible for you link to the source for the plugin to test? In my test environment, I can't download .zip files.

It has to work with any kind of vscode extension, You can try this sample with npm...
Thank you!

@colin-grant-work colin-grant-work merged commit 6179187 into eclipse-theia:master Jul 27, 2021
@vince-fugnitto vince-fugnitto added this to the 1.16.0 milestone Jul 29, 2021
dna2github pushed a commit to dna2fork/theia that referenced this pull request Aug 25, 2021
…se-theia#9780)

Fixes an import of the `setTimeout` symbol that was producing unexpected behavior.

Signed-off-by: Esther Perelman <esther.perelman@sap.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

plug-in system issues related to the plug-in system

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Hosted plugin can't be stop or restart

3 participants