Skip to content

Commit 9b009d6

Browse files
author
OpenShift Bot
authored
Merge pull request #2167 from benjaminapetersen/bpetersen/trello/api-groups/build-configs-instantiate
Merged by openshift-bot
2 parents b09dafe + e8fa089 commit 9b009d6

File tree

4 files changed

+14
-13
lines changed

4 files changed

+14
-13
lines changed

Diff for: app/scripts/controllers/buildConfig.js

+2-1
Original file line numberDiff line numberDiff line change
@@ -46,6 +46,7 @@ angular.module('openshiftConsole')
4646

4747
$scope.buildConfigsVersion = APIService.getPreferredVersion('buildconfigs');
4848
$scope.buildsVersion = APIService.getPreferredVersion('builds');
49+
$scope.buildConfigsInstantiateVersion = APIService.getPreferredVersion('buildconfigs/instantiate');
4950

5051
$scope.emptyMessage = "Loading...";
5152

@@ -92,7 +93,7 @@ angular.module('openshiftConsole')
9293
$scope.envVars = _.filter($scope.envVars, 'name');
9394
buildStrategy($scope.updatedBuildConfig).env = keyValueEditorUtils.compactEntries(angular.copy($scope.envVars));
9495
DataService
95-
.update("buildconfigs", $routeParams.buildconfig, $scope.updatedBuildConfig, $scope.projectContext)
96+
.update($scope.buildConfigsVersion, $routeParams.buildconfig, $scope.updatedBuildConfig, $scope.projectContext)
9697
.then(function success() {
9798
NotificationsService.addNotification({
9899
type: "success",

Diff for: app/views/browse/build-config.html

+5-5
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ <h1 class="contains-actions">
88
<!-- Primary Actions -->
99
<button
1010
class="btn btn-default hidden-xs"
11-
ng-if="('buildconfigs/instantiate' | canI : 'create') && !(buildConfig | isBinaryBuild)"
11+
ng-if="(buildConfigsInstantiateVersion | canI : 'create') && !(buildConfig | isBinaryBuild)"
1212
ng-click="startBuild()">
1313
<span ng-if="!(buildConfig | isJenkinsPipelineStrategy)">
1414
Start Build
@@ -27,7 +27,7 @@ <h1 class="contains-actions">
2727
class="dropdown-toggle actions-dropdown-kebab visible-xs-inline"
2828
data-toggle="dropdown"><i class="fa fa-ellipsis-v"></i><span class="sr-only">Actions</span></a>
2929
<ul class="dropdown-menu dropdown-menu-right actions action-button">
30-
<li class="visible-xs-inline" ng-if="('buildconfigs/instantiate' | canI : 'create') && !(buildConfig | isBinaryBuild)">
30+
<li class="visible-xs-inline" ng-if="(buildConfigsInstantiateVersion | canI : 'create') && !(buildConfig | isBinaryBuild)">
3131
<a href="" role="button" ng-click="startBuild()">
3232
<span ng-if="!(buildConfig | isJenkinsPipelineStrategy)">
3333
Start Build
@@ -83,10 +83,10 @@ <h2>No builds.</h2>
8383

8484
<p>
8585
<span ng-if="!buildConfig.spec.strategy.jenkinsPipelineStrategy">
86-
<span ng-if="!('buildconfigs/instantiate' | canI : 'create')">
86+
<span ng-if="!(buildConfigsInstantiateVersion | canI : 'create')">
8787
Builds will create an image from
8888
</span>
89-
<span ng-if="'buildconfigs/instantiate' | canI : 'create'">
89+
<span ng-if="buildConfigsInstantiateVersion | canI : 'create'">
9090
Start a new build to create an image from
9191
</span>
9292
<span ng-if="buildConfig.spec.source.type === 'Git'">
@@ -118,7 +118,7 @@ <h2>No builds.</h2>
118118
<button
119119
class="btn btn-primary btn-lg"
120120
ng-click="startBuild()"
121-
ng-if="('buildconfigs/instantiate' | canI : 'create') && !(buildConfig | isBinaryBuild)">
121+
ng-if="(buildConfigsInstantiateVersion | canI : 'create') && !(buildConfig | isBinaryBuild)">
122122
<span ng-if="!(buildConfig | isJenkinsPipelineStrategy)">
123123
Start Build
124124
</span>

Diff for: dist/scripts/scripts.js

+2-2
Original file line numberDiff line numberDiff line change
@@ -5295,7 +5295,7 @@ title: "Builds",
52955295
link: "project/" + n.project + "/browse/builds"
52965296
}), e.breadcrumbs.push({
52975297
title: n.buildconfig
5298-
}), e.buildConfigsVersion = a.getPreferredVersion("buildconfigs"), e.buildsVersion = a.getPreferredVersion("builds"), e.emptyMessage = "Loading...", e.aceLoaded = function(e) {
5298+
}), e.buildConfigsVersion = a.getPreferredVersion("buildconfigs"), e.buildsVersion = a.getPreferredVersion("builds"), e.buildConfigsInstantiateVersion = a.getPreferredVersion("buildconfigs/instantiate"), e.emptyMessage = "Loading...", e.aceLoaded = function(e) {
52995299
var t = e.getSession();
53005300
t.setOption("tabSize", 2), t.setOption("useSoftTabs", !0), e.$blockScrolling = 1 / 0;
53015301
};
@@ -5305,7 +5305,7 @@ e.updatedBuildConfig = angular.copy(t), e.envVars = m(e.updatedBuildConfig).env
53055305
e.compareTriggers = function(e, t) {
53065306
return _.isNumber(e.value) ? -1 : "ConfigChange" === e.value ? -1 : "ConfigChange" === t.value ? 1 : "ImageChange" === e.value ? -1 : "ImageChange" === t.value ? 1 : e.value.localeCompare(t.value);
53075307
}, e.saveEnvVars = function() {
5308-
l.hideNotification("save-bc-env-error"), e.envVars = _.filter(e.envVars, "name"), m(e.updatedBuildConfig).env = d.compactEntries(angular.copy(e.envVars)), i.update("buildconfigs", n.buildconfig, e.updatedBuildConfig, e.projectContext).then(function() {
5308+
l.hideNotification("save-bc-env-error"), e.envVars = _.filter(e.envVars, "name"), m(e.updatedBuildConfig).env = d.compactEntries(angular.copy(e.envVars)), i.update(e.buildConfigsVersion, n.buildconfig, e.updatedBuildConfig, e.projectContext).then(function() {
53095309
l.addNotification({
53105310
type: "success",
53115311
message: "Environment variables for build config " + e.buildConfigName + " were successfully updated."

Diff for: dist/scripts/templates.js

+5-5
Original file line numberDiff line numberDiff line change
@@ -1617,7 +1617,7 @@ angular.module('openshiftConsoleTemplates', []).run(['$templateCache', function(
16171617
"<h1 class=\"contains-actions\">\n" +
16181618
"<div class=\"pull-right dropdown\" ng-if=\"buildConfig\" ng-hide=\"!('buildConfigs' | canIDoAny)\">\n" +
16191619
"\n" +
1620-
"<button class=\"btn btn-default hidden-xs\" ng-if=\"('buildconfigs/instantiate' | canI : 'create') && !(buildConfig | isBinaryBuild)\" ng-click=\"startBuild()\">\n" +
1620+
"<button class=\"btn btn-default hidden-xs\" ng-if=\"(buildConfigsInstantiateVersion | canI : 'create') && !(buildConfig | isBinaryBuild)\" ng-click=\"startBuild()\">\n" +
16211621
"<span ng-if=\"!(buildConfig | isJenkinsPipelineStrategy)\">\n" +
16221622
"Start Build\n" +
16231623
"</span>\n" +
@@ -1632,7 +1632,7 @@ angular.module('openshiftConsoleTemplates', []).run(['$templateCache', function(
16321632
"</button>\n" +
16331633
"<a href=\"\" class=\"dropdown-toggle actions-dropdown-kebab visible-xs-inline\" data-toggle=\"dropdown\"><i class=\"fa fa-ellipsis-v\"></i><span class=\"sr-only\">Actions</span></a>\n" +
16341634
"<ul class=\"dropdown-menu dropdown-menu-right actions action-button\">\n" +
1635-
"<li class=\"visible-xs-inline\" ng-if=\"('buildconfigs/instantiate' | canI : 'create') && !(buildConfig | isBinaryBuild)\">\n" +
1635+
"<li class=\"visible-xs-inline\" ng-if=\"(buildConfigsInstantiateVersion | canI : 'create') && !(buildConfig | isBinaryBuild)\">\n" +
16361636
"<a href=\"\" role=\"button\" ng-click=\"startBuild()\">\n" +
16371637
"<span ng-if=\"!(buildConfig | isJenkinsPipelineStrategy)\">\n" +
16381638
"Start Build\n" +
@@ -1677,10 +1677,10 @@ angular.module('openshiftConsoleTemplates', []).run(['$templateCache', function(
16771677
"<h2>No builds.</h2>\n" +
16781678
"<p>\n" +
16791679
"<span ng-if=\"!buildConfig.spec.strategy.jenkinsPipelineStrategy\">\n" +
1680-
"<span ng-if=\"!('buildconfigs/instantiate' | canI : 'create')\">\n" +
1680+
"<span ng-if=\"!(buildConfigsInstantiateVersion | canI : 'create')\">\n" +
16811681
"Builds will create an image from\n" +
16821682
"</span>\n" +
1683-
"<span ng-if=\"'buildconfigs/instantiate' | canI : 'create'\">\n" +
1683+
"<span ng-if=\"buildConfigsInstantiateVersion | canI : 'create'\">\n" +
16841684
"Start a new build to create an image from\n" +
16851685
"</span>\n" +
16861686
"<span ng-if=\"buildConfig.spec.source.type === 'Git'\">\n" +
@@ -1705,7 +1705,7 @@ angular.module('openshiftConsoleTemplates', []).run(['$templateCache', function(
17051705
"</span>\n" +
17061706
"</span>\n" +
17071707
"</p>\n" +
1708-
"<button class=\"btn btn-primary btn-lg\" ng-click=\"startBuild()\" ng-if=\"('buildconfigs/instantiate' | canI : 'create') && !(buildConfig | isBinaryBuild)\">\n" +
1708+
"<button class=\"btn btn-primary btn-lg\" ng-click=\"startBuild()\" ng-if=\"(buildConfigsInstantiateVersion | canI : 'create') && !(buildConfig | isBinaryBuild)\">\n" +
17091709
"<span ng-if=\"!(buildConfig | isJenkinsPipelineStrategy)\">\n" +
17101710
"Start Build\n" +
17111711
"</span>\n" +

0 commit comments

Comments
 (0)