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

Cannot reload after updating extension #693

Closed
UdoW opened this issue Oct 20, 2017 · 8 comments
Closed

Cannot reload after updating extension #693

UdoW opened this issue Oct 20, 2017 · 8 comments
Labels
bug bugs found in the application extension system issues related to the extension system

Comments

@UdoW
Copy link

UdoW commented Oct 20, 2017

I cloned the master GIT repo and followed the build instructions for the browser app. Everything worked fine. I am under Windows 10.

I loaded localhost:3000 and opened the extensions view. Then I updated the marker extension and after reloading I got an empty black screen.

See attached screen shots and log.

updating_marker_extension.log
fresh_start_extensions
after_pressing_update_on_markers_extension
reload_message_appears
reload_pressed

@akosyakov
Copy link
Member

@UdoW Thank you for the issue. Could you also look into the console and post errors from there, please?

@UdoW
Copy link
Author

UdoW commented Oct 20, 2017

I added the log file. Where else can I find logging output?

@akosyakov
Copy link
Member

In Chrome there is More Tools -> Developer Tools and then Console

@UdoW
Copy link
Author

UdoW commented Oct 20, 2017

Ok, I have to rebuild it again...

@akosyakov akosyakov added bug bugs found in the application extension system issues related to the extension system labels Oct 20, 2017
@UdoW
Copy link
Author

UdoW commented Oct 20, 2017

Chrome logs after reloading browser app:

index.js:44 Failed to start the frontend application.
module.exports.Promise.resolve.then.then.then.then.then.then.then.then.then.then.then.then.then.then.then.then.then.then.catch.reason @ index.js:44
Promise rejected (async)
Object.defineProperty.value @ index.js:43
webpack_require @ bootstrap 51292c9bec59add07e65:688
Object.defineProperty.value @ bootstrap 51292c9bec59add07e65:789
(anonymous) @ bootstrap 51292c9bec59add07e65:789
index.js:46 Error: Cannot apply @Injectable decorator multiple times.
at injectable.js:8
at DecorateConstructor (Reflect.ts:1090)
at Object.decorate (Reflect.ts:284)
at _decorate (decorator_utils.js:42)
at Object.decorate (decorator_utils.js:55)
at Object.defineProperty.value (widget.ts:13)
at webpack_require (bootstrap 51292c9bec59add07e65:688)
at fn (bootstrap 51292c9bec59add07e65:109)
at Object.defineProperty.value (index.ts:8)
at webpack_require (bootstrap 51292c9bec59add07e65:688)

@akosyakov
Copy link
Member

@UdoW thank you

In the Theia repo using the extension manager is not going to work for everything since published extensions are outdated compared to the local.

The issue is here that we don't catch cases when frontend bindings are not compatible and because of it cannot revert installation.

It also can be caused by #667

@kittaakos
Copy link
Contributor

This issue still present if one builds an electron application with @next version.

index.js:43 Failed to start the frontend application.
module.exports.Promise.resolve.then.then.then.then.then.then.then.then.then.then.then.then.then.then.then.then.then.catch.reason @ index.js:43
index.js:45 Error: Cannot apply @injectable decorator multiple times.
    at injectable.js:8
    at DecorateConstructor (Reflect.ts:1090)
    at Object.decorate (Reflect.ts:284)
    at _decorate (decorator_utils.js:42)
    at Object.decorate (decorator_utils.js:55)
    at Object.defineProperty.value (widget.ts:13)
    at __webpack_require__ (bootstrap 6b7c3be6c45bdb6cd747:659)
    at fn (bootstrap 6b7c3be6c45bdb6cd747:85)
    at Object.c (index.ts:8)
    at __webpack_require__ (bootstrap 6b7c3be6c45bdb6cd747:659)
module.exports.Promise.resolve.then.then.then.then.then.then.then.then.then.then.then.then.then.then.then.then.then.catch.reason @ index.js:45

@vince-fugnitto
Copy link
Member

Closing since the extension manager has been deprecated #4876

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 extension system issues related to the extension system
Projects
None yet
Development

No branches or pull requests

4 participants