-
Notifications
You must be signed in to change notification settings - Fork 232
Issue labelbinding #1943
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
Issue labelbinding #1943
Conversation
app/scripts/constants.js
Outdated
template_service_broker: false, | ||
pod_presets: false | ||
template_service_broker: true, | ||
pod_presets: true |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should set these back to false in the PR so they're not enabled by default.
app/views/overview.html
Outdated
@@ -358,7 +358,7 @@ <h2 ng-if="!app"> | |||
</overview-list-row> | |||
</div> | |||
</div> | |||
|
|||
<h2>HELLO WORLD!</h2> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Probably want to remove that :)
4976122
to
ad15d47
Compare
ad15d47
to
5d2e2a5
Compare
@spadgett I have made the requested changes :) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM thanks @amarie401
[merge][severity: bug] |
Evaluated for origin web console merge up to 5d2e2a5 |
Origin Web Console Merge Results: SUCCESS (https://ci.openshift.redhat.com/jenkins/job/merge_pull_request_origin_web_console/78/) (Base Commit: bf2a6f2) (PR Branch Commit: 5d2e2a5) (Extended Tests: bug) |
Changed the label for 'Applications Bindings' to just 'Bindings' for expanded Provisioned services. Closes #1939
@ncameronbritt @spadgett