From 21a8091a702127f5ce3c2d2afba686b88ab50d08 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=C5=81ukasz=20Mierzwa?= <l.mierzwa@gmail.com>
Date: Sat, 19 Aug 2017 14:19:53 -0700
Subject: [PATCH] Handle complex annotations correctly in the frontend template

---
 assets/templates/alertgroup.html | 46 +++++++++++++++++---------------
 1 file changed, 25 insertions(+), 21 deletions(-)

diff --git a/assets/templates/alertgroup.html b/assets/templates/alertgroup.html
index b524d00c6..e7e31bcc2 100644
--- a/assets/templates/alertgroup.html
+++ b/assets/templates/alertgroup.html
@@ -27,28 +27,32 @@
 </script>
 
 <script type="application/json" id="alert-group-annotations">
-  <% _.each(sortMapByKey(alert.annotations), function(annotation) { %>
-    <div class="well well-sm annotation-well">
-      <i class="fa fa-info-circle text-muted" title="<%- annotation.key %>" data-toggle="tooltip" data-placement="top"/>
-      <% if (annotation.value) { %>
-        <%= linkify(_.escape(annotation.value)) %>
-      <% } else { %>
-        <span class="text-muted">
-          [ missing annotation value ]
-        </span>
-      <% } %>
-    </div>
+  <% _.each(alert.annotations, function(annotation) { %>
+    <% if (annotation.isLink === false) { %>
+      <div class="well well-sm annotation-well">
+        <i class="fa fa-info-circle text-muted" title="<%- annotation.name %>" data-toggle="tooltip" data-placement="top"/>
+        <% if (annotation.value) { %>
+          <%= linkify(_.escape(annotation.value)) %>
+        <% } else { %>
+          <span class="text-muted">
+            [ missing annotation value ]
+          </span>
+        <% } %>
+      </div>
+    <% } %>
   <% }) %>
-  <% _.each(alert.links, function(url, text) { %>
-    <a class="label label-list label-default"
-       href="<%= url %>"
-       target="_blank"
-       title="<%= url %>"
-       data-toggle="tooltip"
-       data-placement="top">
-      <i class="fa fa-external-link"/>
-      <%- text %>
-    </a>
+  <% _.each(alert.annotations, function(annotation) { %>
+    <% if (annotation.isLink) { %>
+      <a class="label label-list label-default"
+         href="<%= annotation.value %>"
+         target="_blank"
+         title="<%= annotation.value %>"
+         data-toggle="tooltip"
+         data-placement="top">
+        <i class="fa fa-external-link"/>
+        <%- annotation.name %>
+      </a>
+    <% } %>
   <% }) %>
 </script>