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-3209 add import and configure batch of projects on stack testing #3445

Merged
merged 3 commits into from
Jan 10, 2017

Conversation

olexii4
Copy link
Contributor

@olexii4 olexii4 commented Dec 20, 2016

What does this PR do?

Add import and configure batch of projects on stack testing

What issues does this PR fix or reference?

#3209

selection_051
selection_053

Signed-off-by: Oleksii Orel oorel@codenvy.com

@@ -71,9 +71,10 @@ export class SelectTemplateController {
*/
startTest(): void {
let stack: che.IStack = angular.copy(this.stack);
/* tslint:disable */
Copy link
Contributor

Choose a reason for hiding this comment

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

question: why do we need to disable tslint ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@@ -101,7 +103,16 @@ export class StackController {

$window.addEventListener('message', (event: {data: string}) => {
if ('show-ide' === event.data) {
if (this.tmpWorkspaceId) {
//TODO remove this delay after fixing time of update tree
Copy link
Contributor

Choose a reason for hiding this comment

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

is there an open github issue ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Not yet. It is a time for project type update(from API). I should investigate it more.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

fixed

* @param index{number} - the index of the array of commands
* @param deferred{ng.IDeferred<any>}
*/
updateProjects(workspaceId: string, projects: Array<any>, index: number, deferred: ng.IDeferred<any>) {
Copy link
Contributor

Choose a reason for hiding this comment

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

return type is void ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes. I will fix

Copy link
Contributor Author

Choose a reason for hiding this comment

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

fixed

while (iterations--) {
name = 'wksp-' + (('0000' + (Math.random() * Math.pow(36, 4) << 0).toString(36)).slice(-4)); // jshint ignore:line
if (!this.usedNamesList.indexOf(name) >= 0) {
/* tslint:disable */
Copy link
Contributor

Choose a reason for hiding this comment

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

same question than before ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@codenvy-ci
Copy link

Build # 1401 - FAILED

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

@olexii4 olexii4 force-pushed the CHE-3209 branch 2 times, most recently from 377da41 to fe273a0 Compare January 4, 2017 12:40
@codenvy-ci
Copy link

Oleksii Orel added 3 commits January 10, 2017 18:06
Signed-off-by: Oleksii Orel <oorel@codenvy.com>
Signed-off-by: Oleksii Orel <oorel@codenvy.com>
Signed-off-by: Oleksii Orel <oorel@codenvy.com>
@olexii4 olexii4 merged commit bc571b6 into master Jan 10, 2017
@olexii4 olexii4 deleted the CHE-3209 branch January 10, 2017 16:10
@slemeur slemeur added the kind/enhancement A feature request - must adhere to the feature request template. label Jan 18, 2017
@slemeur slemeur added this to the 5.1.0 milestone Jan 18, 2017
@JamesDrummond JamesDrummond mentioned this pull request Jan 31, 2017
9 tasks
JPinkney pushed a commit to JPinkney/che that referenced this pull request Aug 17, 2017
CHE-3209 add import and configure batch of projects on stack testing
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.

5 participants