Commit e1b739d0 by Jessica Forrester Committed by GitHub

Merge pull request #40 from spadgett/fix-template-descriptions

Bug 1444200 - Fix problem showing template descriptions
parents b079b719 0dabd40c
...@@ -868,8 +868,10 @@ angular.module('openshiftCommonUI') ...@@ -868,8 +868,10 @@ angular.module('openshiftCommonUI')
.filter('description', function(annotationFilter) { .filter('description', function(annotationFilter) {
return function(resource) { return function(resource) {
// Prefer `openshift.io/description`, but fall back to `kubernetes.io/description`. // Prefer `openshift.io/description`, but fall back to `kubernetes.io/description`.
// Templates use simply `description` without a namespace.
return annotationFilter(resource, 'openshift.io/description') || return annotationFilter(resource, 'openshift.io/description') ||
annotationFilter(resource, 'kubernetes.io/description'); annotationFilter(resource, 'kubernetes.io/description') ||
annotationFilter(resource, 'description');
}; };
}) })
.filter('displayName', function(annotationFilter) { .filter('displayName', function(annotationFilter) {
......
...@@ -1075,8 +1075,10 @@ angular.module('openshiftCommonUI') ...@@ -1075,8 +1075,10 @@ angular.module('openshiftCommonUI')
.filter('description', ["annotationFilter", function(annotationFilter) { .filter('description', ["annotationFilter", function(annotationFilter) {
return function(resource) { return function(resource) {
// Prefer `openshift.io/description`, but fall back to `kubernetes.io/description`. // Prefer `openshift.io/description`, but fall back to `kubernetes.io/description`.
// Templates use simply `description` without a namespace.
return annotationFilter(resource, 'openshift.io/description') || return annotationFilter(resource, 'openshift.io/description') ||
annotationFilter(resource, 'kubernetes.io/description'); annotationFilter(resource, 'kubernetes.io/description') ||
annotationFilter(resource, 'description');
}; };
}]) }])
.filter('displayName', ["annotationFilter", function(annotationFilter) { .filter('displayName', ["annotationFilter", function(annotationFilter) {
......
This source diff could not be displayed because it is too large. You can view the blob instead.
...@@ -26,8 +26,10 @@ angular.module('openshiftCommonUI') ...@@ -26,8 +26,10 @@ angular.module('openshiftCommonUI')
.filter('description', function(annotationFilter) { .filter('description', function(annotationFilter) {
return function(resource) { return function(resource) {
// Prefer `openshift.io/description`, but fall back to `kubernetes.io/description`. // Prefer `openshift.io/description`, but fall back to `kubernetes.io/description`.
// Templates use simply `description` without a namespace.
return annotationFilter(resource, 'openshift.io/description') || return annotationFilter(resource, 'openshift.io/description') ||
annotationFilter(resource, 'kubernetes.io/description'); annotationFilter(resource, 'kubernetes.io/description') ||
annotationFilter(resource, 'description');
}; };
}) })
.filter('displayName', function(annotationFilter) { .filter('displayName', function(annotationFilter) {
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment