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

Add commands to workspace when importing project samples #2976

Merged
merged 3 commits into from
Jan 27, 2017

Conversation

kaloyan-raev
Copy link
Contributor

@kaloyan-raev kaloyan-raev commented Nov 3, 2016

What does this PR do?

If a project sample is imported from the IDE's Create Project wizard, the commands defined in the project sample will be added to the workspace. This way it will be the same like importing the project sample from the Dashboard.

What issues does this PR fix or reference?

None.

Previous behavior

Commands defined in the project sample were not added to the workspace.

New behavior

The commands are now added.

Changelog

Add commands to workspace when importing project samples

Signed-off-by: Kaloyan Raev [email protected]

@codenvy-ci
Copy link

Can one of the admins verify this patch?

@benoitf
Copy link
Contributor

benoitf commented Nov 3, 2016

ci-build

@benoitf benoitf added kind/enhancement A feature request - must adhere to the feature request template. status/code-review This issue has a pull request posted for it and is awaiting code review completion by the community. team/ide labels Nov 3, 2016
@slemeur
Copy link
Contributor

slemeur commented Nov 3, 2016

+1

@codenvy-ci
Copy link

Build # 917 - FAILED

Please check console output at https://ci.codenvycorp.com/job/che-pullrequests-build/917/ to view the results.

@benoitf
Copy link
Contributor

benoitf commented Nov 8, 2016

@vparfonov ?

@TylerJewell
Copy link

@kaloyan-raev - lets fix the conflicting files. I'm asking Vitalli to take this into next sprint for merge. I'm going to ask all TLs to go through PR backlog now that we are 5.0.0 shipped.

Conflicts:
	ide/che-core-ide-api/src/main/java/org/eclipse/che/ide/api/project/MutableProjectConfig.java
	ide/che-core-ide-app/src/main/java/org/eclipse/che/ide/projecttype/wizard/presenter/ProjectWizardPresenter.java
@kaloyan-raev
Copy link
Contributor Author

Conflicts resolved. Rebased to latest master.

@vparfonov vparfonov added this to the 5.2.0 milestone Jan 18, 2017
@vparfonov vparfonov self-assigned this Jan 18, 2017
@riuvshin
Copy link
Contributor

@vparfonov what is the state of that issue? it is planned for 5.2.0 which going tobe released in few days.

@vparfonov vparfonov merged commit a652db2 into eclipse-che:master Jan 27, 2017
@JamesDrummond JamesDrummond removed status/code-review This issue has a pull request posted for it and is awaiting code review completion by the community. sprint/current labels Feb 3, 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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/enhancement A feature request - must adhere to the feature request template.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants