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

IDEX-4228: Fix import project from ssh url #318

Merged
merged 1 commit into from
Feb 16, 2016
Merged

IDEX-4228: Fix import project from ssh url #318

merged 1 commit into from
Feb 16, 2016

Conversation

vinokurig
Copy link
Contributor

Signed-off-by: Igor Vinokur ivinokur@codenvy.com
When importing project by ssh url without ssh keys, empty project appears after refreshing the page. Also the error message was ugly.
@skabashnyuk @vparfonov please review

@skabashnyuk
Copy link
Contributor

ok @vparfonov ?

String errorMessage = ErrorMessageUtils.getErrorMessage(exception.getCause());
if (errorMessage.equals("Unable get private ssh key")) {
subscriber.onFailure(errorMessage);
if ("Unable get private ssh key".equals(errorMessage)) {
Copy link
Contributor

Choose a reason for hiding this comment

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

Should be changed to errorCode then it will be done

@vparfonov
Copy link
Contributor

ok

@vinokurig vinokurig changed the title IDEX-2428: Fix import project from ssh url IDEX-4228: Fix import project from ssh url Feb 15, 2016
Signed-off-by: Igor Vinokur <ivinokur@codenvy.com>
vinokurig pushed a commit that referenced this pull request Feb 16, 2016
IDEX-4228: Fix import project from ssh url
@vinokurig vinokurig merged commit 8e25c75 into master Feb 16, 2016
@vinokurig vinokurig deleted the IDEX-4228 branch February 16, 2016 13:28
Katka92 pushed a commit to Katka92/che that referenced this pull request Jun 27, 2018
…ching. Reactivating without dashboard build (eclipse-che#318)

Signed-off-by: Sun Tan <sutan@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants