Skip to content

Commit b647c5c

Browse files
committed
Bug 1388447 - On BC editor page, when 'Push To' is 'None', should disable 'Push Secret'
1 parent 34067a1 commit b647c5c

File tree

5 files changed

+10
-7
lines changed

5 files changed

+10
-7
lines changed

app/scripts/directives/oscSecrets.js

+1
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,7 @@ angular.module("openshiftConsole")
1212
displayType: "@",
1313
type: "@",
1414
alerts: '=',
15+
disableInput: "=",
1516
serviceAccountToLink: '@?',
1617
allowMultipleSecrets: "=?" // false by default
1718
},

app/views/directives/osc-secrets.html

+3-3
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@
55
<div class="col-lg-12">
66
<div ng-if="!allowMultipleSecrets">
77
<label class="picker-label">{{displayType | startCase}} Secret</label>
8-
<ui-select ng-model="pickedSecret.name">
8+
<ui-select ng-disabled="disableInput" ng-model="pickedSecret.name">
99
<ui-select-match placeholder="Secret name">{{$select.selected}}</ui-select-match>
1010
<ui-select-choices repeat="secret in (secretsByType[type] | filter : $select.search)">
1111
<div ng-bind-html="secret | highlight : $select.search"></div>
@@ -16,7 +16,7 @@
1616
<div class="basic-secrets">
1717
<div class="secret-name">
1818
<label ng-if="$first" class="picker-label">{{displayType | startCase}} Secrets</label>
19-
<ui-select ng-model="pickedSecret.name">
19+
<ui-select ng-disabled="disableInput" ng-model="pickedSecret.name">
2020
<ui-select-match placeholder="Secret name">{{$select.selected}}</ui-select-match>
2121
<ui-select-choices repeat="secret in (secretsByType[type] | filter : $select.search)">
2222
<div ng-bind-html="secret | highlight : $select.search"></div>
@@ -48,7 +48,7 @@
4848
</div>
4949
</div>
5050

51-
<div class="osc-secret-actions">
51+
<div class="osc-secret-actions" ng-if="!disableInput">
5252
<span ng-if="canAddSourceSecret()">
5353
<a href=""
5454
role="button"

app/views/edit/build-config.html

+1
Original file line numberDiff line numberDiff line change
@@ -410,6 +410,7 @@ <h3 class="with-divider">Image Configuration</h3>
410410
namespace="projectName"
411411
display-type="push"
412412
type="image"
413+
disable-input="imageOptions.to.type==='None'"
413414
service-account-to-link="builder"
414415
secrets-by-type="secrets.secretsByType"
415416
alerts="alerts">

dist/scripts/scripts.js

+1
Original file line numberDiff line numberDiff line change
@@ -9149,6 +9149,7 @@ namespace:"=",
91499149
displayType:"@",
91509150
type:"@",
91519151
alerts:"=",
9152+
disableInput:"=",
91529153
serviceAccountToLink:"@?",
91539154
allowMultipleSecrets:"=?"
91549155
},

dist/scripts/templates.js

+4-4
Original file line numberDiff line numberDiff line change
@@ -7012,7 +7012,7 @@ angular.module('openshiftConsoleTemplates', []).run(['$templateCache', function(
70127012
"<div class=\"col-lg-12\">\n" +
70137013
"<div ng-if=\"!allowMultipleSecrets\">\n" +
70147014
"<label class=\"picker-label\">{{displayType | startCase}} Secret</label>\n" +
7015-
"<ui-select ng-model=\"pickedSecret.name\">\n" +
7015+
"<ui-select ng-disabled=\"disableInput\" ng-model=\"pickedSecret.name\">\n" +
70167016
"<ui-select-match placeholder=\"Secret name\">{{$select.selected}}</ui-select-match>\n" +
70177017
"<ui-select-choices repeat=\"secret in (secretsByType[type] | filter : $select.search)\">\n" +
70187018
"<div ng-bind-html=\"secret | highlight : $select.search\"></div>\n" +
@@ -7023,7 +7023,7 @@ angular.module('openshiftConsoleTemplates', []).run(['$templateCache', function(
70237023
"<div class=\"basic-secrets\">\n" +
70247024
"<div class=\"secret-name\">\n" +
70257025
"<label ng-if=\"$first\" class=\"picker-label\">{{displayType | startCase}} Secrets</label>\n" +
7026-
"<ui-select ng-model=\"pickedSecret.name\">\n" +
7026+
"<ui-select ng-disabled=\"disableInput\" ng-model=\"pickedSecret.name\">\n" +
70277027
"<ui-select-match placeholder=\"Secret name\">{{$select.selected}}</ui-select-match>\n" +
70287028
"<ui-select-choices repeat=\"secret in (secretsByType[type] | filter : $select.search)\">\n" +
70297029
"<div ng-bind-html=\"secret | highlight : $select.search\"></div>\n" +
@@ -7054,7 +7054,7 @@ angular.module('openshiftConsoleTemplates', []).run(['$templateCache', function(
70547054
"</div>\n" +
70557055
"</div>\n" +
70567056
"</div>\n" +
7057-
"<div class=\"osc-secret-actions\">\n" +
7057+
"<div class=\"osc-secret-actions\" ng-if=\"!disableInput\">\n" +
70587058
"<span ng-if=\"canAddSourceSecret()\">\n" +
70597059
"<a href=\"\" role=\"button\" ng-click=\"addSourceSecret()\">Add another secret</a>\n" +
70607060
"<span ng-if=\"'secrets' | canI : 'create'\" class=\"action-divider\">|</span>\n" +
@@ -7730,7 +7730,7 @@ angular.module('openshiftConsoleTemplates', []).run(['$templateCache', function(
77307730
"</div>\n" +
77317731
"</div>\n" +
77327732
"<div class=\"form-group\" ng-if=\"view.advancedOptions\">\n" +
7733-
"<osc-secrets model=\"secrets.picked.pushSecret\" namespace=\"projectName\" display-type=\"push\" type=\"image\" service-account-to-link=\"builder\" secrets-by-type=\"secrets.secretsByType\" alerts=\"alerts\">\n" +
7733+
"<osc-secrets model=\"secrets.picked.pushSecret\" namespace=\"projectName\" display-type=\"push\" type=\"image\" disable-input=\"imageOptions.to.type==='None'\" service-account-to-link=\"builder\" secrets-by-type=\"secrets.secretsByType\" alerts=\"alerts\">\n" +
77347734
"</osc-secrets>\n" +
77357735
"</div>\n" +
77367736
"</div>\n" +

0 commit comments

Comments
 (0)