Merge pull request #2721 from jlmitch5/fixJTSaveCompletedDeferring

make credential association cause deferral of jt promise chain
This commit is contained in:
John Mitchell
2018-08-02 09:45:19 -04:00
committed by GitHub
2 changed files with 3 additions and 3 deletions

View File

@@ -500,7 +500,7 @@ export default
null, true);
}
MultiCredentialService
var credDefer = MultiCredentialService
.saveRelated(jobTemplateData, $scope.multiCredential.selectedCredentials);
InstanceGroupsService.editInstanceGroups(instance_group_url, $scope.instance_groups)
@@ -580,7 +580,7 @@ export default
Rest.setUrl(data.related.labels);
var defers = [];
var defers = [credDefer];
for (var i = 0; i < toPost.length; i++) {
defers.push(Rest.post(toPost[i]));
}

View File

@@ -40,7 +40,7 @@ function MultiCredentialService (Rest, ProcessErrors, $q, GetBasePath) {
return $q.all(disassociationPromises).then(() => {
_.each(selected.filter(id => currentlyAssociated.indexOf(id) < 0), (id) => {
associate({related}, id);
return associate({related}, id);
});
});
});