From 6ad34edc5e0f155f73032dcb2f015f82f94b296c Mon Sep 17 00:00:00 2001
From: Mat <MatMoore@users.noreply.github.com>
Date: Mon, 4 Nov 2024 10:45:11 +0000
Subject: [PATCH] workaround: refer to data owners in Datahub as custodians in
 Find MoJ data (#1014)

fix: refer to data owners at custodians

This is a temporary workaround. Once the next version of Datahub is
released, we will be able to reenable the `meta_mapping` in the dbt
ingestion, which will allow us to store Data custodians as Data
custodians.

At this point we will be able to distinguish between
different kinds of owners in Datahub, and we can remove this workaround.

The ability to distinguish between owners and custodians is not
essential at the moment, but it's desirable, because the data improvement
team are interesting data owner/data steward/data custodian roles, and
it's very likely that data owners and stewards will also interact with
the Find MoJ data and/or datahub.
---
 templates/partial/contact_info.html | 12 ++++--------
 1 file changed, 4 insertions(+), 8 deletions(-)

diff --git a/templates/partial/contact_info.html b/templates/partial/contact_info.html
index 6ffe46e7..b34f9f3f 100644
--- a/templates/partial/contact_info.html
+++ b/templates/partial/contact_info.html
@@ -9,10 +9,8 @@ <h2 class="govuk-heading-s govuk-!-margin-bottom-1">{% translate "Access require
       {% else %}
         {{ access_requirements }}
       {% endif %}
-    {% elif governance.data_custodians %}
+    {% elif governance.data_custodians or governance.data_owner.email %}
       {% translate "Please contact the data custodian for access information." %}
-    {% elif governance.data_owner.email %}
-      {% translate "Please contact the data owner for access information." %}
     {% else %}
       {% translate "Not provided" %}
     {% endif %}
@@ -36,12 +34,10 @@ <h2 class="govuk-heading-s govuk-!-margin-bottom-1">{% translate "Contact channe
     {% if further_information.dc_team_email %}
       <li id="contact_channels_team_email" >{{ further_information.dc_team_email|urlize }}</li>
     {% endif %}
-    
+
     {% if not further_information.dc_teams_channel_url and not further_information.dc_slack_channel_url and not further_information.dc_team_email %}
-      {% if governance.data_custodians %}
+      {% if governance.data_custodians or governance.data_owner.email %}
       <li id="contact_channels_data_owner">{% translate "Contact the data custodian with questions." %}</li>
-      {% elif governance.data_owner.email %}
-      <li id="contact_channels_data_owner">{% translate "Contact the data owner with questions." %}</li>
       {% else %}
       <li id="contact_channels_not_provided">{% translate "Not provided" %}</li>
       {% endif %}
@@ -58,7 +54,7 @@ <h2 class="govuk-heading-s govuk-!-margin-bottom-1">{% translate "Data custodian
 </div>
 {% else %}
 <div class="govuk-body">
-  <h2 class="govuk-heading-s govuk-!-margin-bottom-1">{% translate "IAO or Data Owner" %}</h2>
+  <h2 class="govuk-heading-s govuk-!-margin-bottom-1">{% translate "Data custodian" %}</h2>
   <p id="data_owner" class="govuk-body">
     {% if governance.data_owner.email %}
       {{ governance.data_owner.email|urlize }}