Skip to content

Commit

Permalink
Merge pull request #41 from smarterclayton/remove_phase_reason
Browse files Browse the repository at this point in the history
Remove the reason from the phase display
  • Loading branch information
openshift-merge-robot committed Mar 11, 2019
2 parents d6d0e02 + be831a0 commit e1a83bd
Showing 1 changed file with 1 addition and 2 deletions.
3 changes: 1 addition & 2 deletions cmd/release-controller/http_helper.go
Expand Up @@ -52,11 +52,10 @@ func phaseCell(tag imagev1.TagReference) string {
phase := tag.Annotations[releaseAnnotationPhase]
switch phase {
case releasePhaseRejected:
return fmt.Sprintf("<td class=\"%s\" title=\"%s\">%s (%s)</td>",
return fmt.Sprintf("<td class=\"%s\" title=\"%s\">%s</td>",
phaseAlert(tag),
template.HTMLEscapeString(tag.Annotations[releaseAnnotationMessage]),
template.HTMLEscapeString(phase),
template.HTMLEscapeString(tag.Annotations[releaseAnnotationReason]),
)
}
return fmt.Sprintf("<td class=\"%s\">", phaseAlert(tag)) + template.HTMLEscapeString(phase) + "</td>"
Expand Down

0 comments on commit e1a83bd

Please sign in to comment.