From 126e93fcc3b37bb40a9b889ffb9f2ce7640c7b5d Mon Sep 17 00:00:00 2001
From: Phil Segal <phil@phisec.co.uk>
Date: Thu, 9 Jan 2025 12:55:15 +0000
Subject: [PATCH] 485837: Add comment about bucket naming

---
 src/server/services/buckets/views/all.njk         | 10 ++++++++++
 src/server/services/buckets/views/environment.njk | 11 +++++++++++
 2 files changed, 21 insertions(+)

diff --git a/src/server/services/buckets/views/all.njk b/src/server/services/buckets/views/all.njk
index 965ac1f0..6ee297c0 100644
--- a/src/server/services/buckets/views/all.njk
+++ b/src/server/services/buckets/views/all.njk
@@ -12,6 +12,11 @@
   <a class="app-link app-link--underline app-link--text-colour" href="{{ supportChannel }}" target="_blank"
      rel="noopener noreferrer">#cdp-support</a>
 {% endset %}
+{% set bucketNamesInfoHtml %}
+  Why do bucket names have random characters at the end?
+  <a class="app-link app-link--underline app-link--text-colour" href="/documentation/how-to/buckets.md#bucket-naming" target="_blank"
+     rel="noopener noreferrer">See Bucket Naming documentation</a> for more details
+{% endset %}
 
 {% set allEnvironmentsTables = [] %}
 {% for environment, detail in bucketsByEnvironment %}
@@ -72,6 +77,11 @@
             View buckets for {{ service.serviceName }} in all CDP environments
           </p>
 
+          {{ appInfo({
+            classes: "govuk-!-margin-bottom-4",
+            html: bucketNamesInfoHtml
+          }) }}
+
           <div class="app-grid app-grid--fluid">
 
             {% for detail in allEnvironmentsTables %}
diff --git a/src/server/services/buckets/views/environment.njk b/src/server/services/buckets/views/environment.njk
index 2968c094..cfa2cd8e 100644
--- a/src/server/services/buckets/views/environment.njk
+++ b/src/server/services/buckets/views/environment.njk
@@ -12,6 +12,12 @@
   <a class="app-link app-link--underline app-link--text-colour" href="{{ supportChannel }}" target="_blank"
      rel="noopener noreferrer">#cdp-support</a>
 {% endset %}
+{% set bucketNamesInfoHtml %}
+  Why do bucket names have random characters at the end?
+  <a class="app-link app-link--underline app-link--text-colour" href="/documentation/how-to/buckets.md#bucket-naming" target="_blank"
+     rel="noopener noreferrer">See Bucket Naming documentation</a> for more details
+{% endset %}
+
 
 {% set bucketsTableRows = [] %}
 {% for bucket in buckets %}
@@ -46,6 +52,11 @@
                   Buckets available to your service
                 </p>
 
+                {{ appInfo({
+                  classes: "govuk-!-margin-bottom-4",
+                  html: bucketNamesInfoHtml
+                }) }}
+
                 {% call appPanel({ classes: "govuk-!-margin-bottom-6" }) %}
                   {% if buckets | length %}
                     {{ govukTable({