Skip to content
This repository has been archived by the owner on Jul 19, 2019. It is now read-only.

Set correct che api external enviroment variable. Use official released workspace client. #2

Merged
merged 6 commits into from
Jun 18, 2018

Conversation

AndrienkoAleksandr
Copy link
Contributor

Set correct che api external env. variable, because pull request: eclipse-che/che#9475 was merged.
Use official released Eclipse CHE workspace client.
Set skip lib checks to prevent fail build.

@@ -1,5 +1,6 @@
{
"compilerOptions": {
"skipLibCheck": true,
Copy link
Member

Choose a reason for hiding this comment

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

why do we need to skip the check ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Without skip check lib build fails.

…heck, use strict js.

Signed-off-by: Oleksandr Andriienko <oandriie@redhat.com>
Apply yarn.lock for reproducible build.
Fix incorrect list machines when user open machine panel and do browser page refresh.
Synchronize getting workspaceId and che api end point.
Code clean up.
Update version of @eclipse-che/workspace-client

Signed-off-by: Oleksandr Andriienko <oandriie@redhat.com>
Signed-off-by: Oleksandr Andriienko <oandriie@redhat.com>
Signed-off-by: Oleksandr Andriienko <oandriie@redhat.com>
@AndrienkoAleksandr
Copy link
Contributor Author

AndrienkoAleksandr commented Jun 18, 2018

@olexii4, @mmorhun take a look again, please.
I applied yarn.lock file,
fixed synchronization bug when plugin can get workspaceId or api end point earlier than it was initialized, fixed bug when list machines contains duplication when user open machine panel and refresh browser,
used Env variable service from theia-core.

@AndrienkoAleksandr AndrienkoAleksandr merged commit 408f747 into eclipse:master Jun 18, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants