update remote site delete ui text labels (#1349)

bihealth · Feb 9, 2024 · 2b2bcf7 · 2b2bcf7
1 parent f84a0f8
commit 2b2bcf7
Show file tree

Hide file tree

Showing 3 changed files with 12 additions and 6 deletions.
diff --git 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
@@ -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
@@ -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>