Skip to content

Commit

Permalink
fix: refer to data owners at custodians
Browse files Browse the repository at this point in the history
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.
  • Loading branch information
MatMoore committed Nov 4, 2024
1 parent c8f88ed commit 9244159
Showing 1 changed file with 4 additions and 8 deletions.
12 changes: 4 additions & 8 deletions templates/partial/contact_info.html
Original file line number Diff line number Diff line change
Expand Up @@ -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 %}
Expand All @@ -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 %}
Expand All @@ -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 }}
Expand Down

0 comments on commit 9244159

Please sign in to comment.