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

CHE-3797 fix project import in case when workspace is running #3898

Merged
merged 1 commit into from
Jan 26, 2017

Conversation

olexii4
Copy link
Contributor

@olexii4 olexii4 commented Jan 25, 2017

Signed-off-by: Oleksii Orel [email protected]

What does this PR do?

Fix project import in case when workspace is running.

What issues does this PR fix or reference?

#3797

Changelog and Release Note Information

Changelog

Fix bug with infinite expectation during project import in case with running workspace.

Release notes:

  • Fix bug with infinite expectation during project import in case with running workspace.
  • Add possibility to catch websocket url error for target workspace before project import.

@benoitf benoitf added kind/bug Outline of a bug - must adhere to the bug report template. status/code-review This issue has a pull request posted for it and is awaiting code review completion by the community. labels Jan 25, 2017
@@ -177,7 +177,7 @@ export class CheWorkspace {
});
}

getWorkspacesByNamespace(namespace) {
getWorkspacesByNamespace(namespace: string): che.IWorkspace {
Copy link
Contributor

Choose a reason for hiding this comment

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

Here the list is returned.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

@codenvy-ci
Copy link

@codenvy-ci
Copy link

@ashumilova ashumilova added this to the 5.2.0 milestone Jan 25, 2017
@olexii4 olexii4 merged commit ad96911 into master Jan 26, 2017
@olexii4 olexii4 deleted the CHE-3797 branch January 26, 2017 09:03
@slemeur slemeur removed the status/code-review This issue has a pull request posted for it and is awaiting code review completion by the community. label Jan 26, 2017
@JamesDrummond JamesDrummond mentioned this pull request Feb 6, 2017
9 tasks
JPinkney pushed a commit to JPinkney/che that referenced this pull request Aug 17, 2017
CHE-3797 fix project import in case when workspace is running
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Outline of a bug - must adhere to the bug report template.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants