From 2b2bcf700039b3142f02cdea247ab7a956e2c62d Mon Sep 17 00:00:00 2001
From: Mikko Nieminen <mikko.nieminen@bih-charite.de>
Date: Fri, 9 Feb 2024 16:29:34 +0100
Subject: [PATCH] update remote site delete ui text labels (#1349)

---
 CHANGELOG.rst                                      |  2 ++
 .../projectroles/_remote_site_dropdown.html        |  2 +-
 .../projectroles/remotesite_confirm_delete.html    | 14 +++++++++-----
 3 files changed, 12 insertions(+), 6 deletions(-)

diff --git a/CHANGELOG.rst b/CHANGELOG.rst
index bafc6db9..675e592d 100644
--- a/CHANGELOG.rst
+++ b/CHANGELOG.rst
@@ -20,6 +20,8 @@ Changed
 - **General**
     - Upgrade minimum Django version to v3.2.24 (#1348)
     - Upgrade LDAP dependencies (#1348)
+- **Projectroles**
+    - Improve remote site deletion UI text labels (#1349)
 
 Fixed
 -----
diff --git a/projectroles/templates/projectroles/_remote_site_dropdown.html b/projectroles/templates/projectroles/_remote_site_dropdown.html
index ea8c6810..1b393adc 100644
--- a/projectroles/templates/projectroles/_remote_site_dropdown.html
+++ b/projectroles/templates/projectroles/_remote_site_dropdown.html
@@ -26,6 +26,6 @@
   </a>
   <a class="dropdown-item text-danger"
      href="{% url 'projectroles:remote_site_delete' remotesite=site.sodar_uuid %}">
-    <i class="iconify" data-icon="mdi:close-thick"></i> Remove Site
+    <i class="iconify" data-icon="mdi:close-thick"></i> Delete Site
   </a>
 </div>
diff --git a/projectroles/templates/projectroles/remotesite_confirm_delete.html b/projectroles/templates/projectroles/remotesite_confirm_delete.html
index 914a25f2..34d85b28 100644
--- a/projectroles/templates/projectroles/remotesite_confirm_delete.html
+++ b/projectroles/templates/projectroles/remotesite_confirm_delete.html
@@ -4,23 +4,27 @@
 {% load crispy_forms_filters %}
 {% load projectroles_common_tags %}
 
-{% block title %}Confirm Removal of Remote Site{% endblock %}
+{% block title %}Confirm Deletion of Remote Site{% endblock %}
 
 {% block projectroles %}
 
 {% get_django_setting 'PROJECTROLES_SITE_MODE' as site_mode %}
 
 <div class="row sodar-subtitle-container">
-  <h2>Confirm Removal of Remote Site</h2>
+  <h2>Confirm Deletion of Remote Site</h2>
 </div>
 
 <div class="container-fluid sodar-page-container">
   <div class="alert alert-warning" role="alert">
-    Do you really want to remove the remote {{ object.mode | lower }} site
+    Do you really want to remove the delete {{ object.mode | lower }} site
     "<strong>{{ object.name }}</strong>"? Please note that this will also
-    remove all {% get_display_name 'PROJECT' %} access rules associated
+    delete all {% get_display_name 'PROJECT' %} access rules associated
     with the site!
   </div>
+  <div class="alert alert-warning" role="alert">
+    If the remote site is still in use, once its entry is deleted it will no
+    longer be able to sync projects to/from this site.
+  </div>
 
   <form method="post">
     {% csrf_token %}
@@ -30,7 +34,7 @@ <h2>Confirm Removal of Remote Site</h2>
         <i class="iconify" data-icon="mdi:arrow-left-circle"></i> Cancel
       </a>
       <button type="submit" class="btn btn-danger">
-        <i class="iconify" data-icon="mdi:close-thick"></i> Remove
+        <i class="iconify" data-icon="mdi:close-thick"></i> Delete
       </button>
     </div>
   </form>