summaryrefslogtreecommitdiffstats
path: root/controller-server
diff options
context:
space:
mode:
authorØyvind Grønnesby <oyving@yahooinc.com>2022-12-20 15:34:24 +0100
committerØyvind Grønnesby <oyving@yahooinc.com>2022-12-20 15:34:24 +0100
commit79b8d5c7587d8f4727686029c53f07afa1ec177b (patch)
tree92e9dc00c70d7129d73277e3dd7be94941e2814c /controller-server
parent2d82ff0d3a0f95a3b19c61e318a3cb3dfa6a2491 (diff)
Small notification tweaks
Diffstat (limited to 'controller-server')
-rw-r--r--controller-server/src/main/java/com/yahoo/vespa/hosted/controller/notification/Notifier.java2
-rw-r--r--controller-server/src/main/resources/mail/mail-notification.tmpl8
-rw-r--r--controller-server/src/test/resources/mail/notification.txt8
3 files changed, 7 insertions, 11 deletions
diff --git a/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/notification/Notifier.java b/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/notification/Notifier.java
index 5d0e71b0727..aeae65ad9d2 100644
--- a/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/notification/Notifier.java
+++ b/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/notification/Notifier.java
@@ -136,7 +136,7 @@ public class Notifier {
.replace("[[NOTIFICATION_HEADER]]", content.messagePrefix())
.replace("[[NOTIFICATION_ITEMS]]", notification.messages().stream()
.map(Notifier::linkify)
- .map(m -> "<li>" + m + "</li>")
+ .map(m -> "<p>" + m + "</p>")
.collect(Collectors.joining()))
.replace("[[LINK_TO_NOTIFICATION]]", notificationLink(notification.source()))
.replace("[[LINK_TO_ACCOUNT_NOTIFICATIONS]]", accountNotificationsUri(content.notification().source().tenant()))
diff --git a/controller-server/src/main/resources/mail/mail-notification.tmpl b/controller-server/src/main/resources/mail/mail-notification.tmpl
index 8c02c05723f..5bf5530b433 100644
--- a/controller-server/src/main/resources/mail/mail-notification.tmpl
+++ b/controller-server/src/main/resources/mail/mail-notification.tmpl
@@ -414,7 +414,7 @@
line-height: 32px;
"
>
- Vespa cloud notifications
+ Vespa Cloud Notifications
</h1>
</div>
</td>
@@ -443,11 +443,9 @@
"
>
<p>
- [[NOTIFICATION_HEADER]]
+ [[NOTIFICATION_HEADER]]:
</p>
- <ul>
- [[NOTIFICATION_ITEMS]]
- </ul>
+ [[NOTIFICATION_ITEMS]]
</div>
</td>
</tr>
diff --git a/controller-server/src/test/resources/mail/notification.txt b/controller-server/src/test/resources/mail/notification.txt
index 9f1583d7674..946d9c5fd68 100644
--- a/controller-server/src/test/resources/mail/notification.txt
+++ b/controller-server/src/test/resources/mail/notification.txt
@@ -414,7 +414,7 @@
line-height: 32px;
"
>
- Vespa cloud notifications
+ Vespa Cloud Notifications
</h1>
</div>
</td>
@@ -443,11 +443,9 @@
"
>
<p>
- There are problems with tests for default.default
+ There are problems with tests for default.default:
</p>
- <ul>
- <li>test package has production tests, but no production tests are declared in deployment.xml</li><li>see <a href="https://docs.vespa.ai/en/testing.html">https://docs.vespa.ai/en/testing.html</a> for details on how to write system tests for Vespa</li>
- </ul>
+ <p>test package has production tests, but no production tests are declared in deployment.xml</p><p>see <a href="https://docs.vespa.ai/en/testing.html">https://docs.vespa.ai/en/testing.html</a> for details on how to write system tests for Vespa</p>
</div>
</td>
</tr>