Don't show some enpty fields.

cedadev · Nov 19, 2024 · d732771 · d732771
1 parent efc9620
commit d732771
Showing 1 changed file with 4 additions and 0 deletions.
diff --git a/jasmin_services/templates/jasmin_services/includes/display_accesses.html b/jasmin_services/templates/jasmin_services/includes/display_accesses.html
@@ -83,12 +83,16 @@
                 <p>{{ access.user_reason|markdown }} </p>
                 {% if for_managers %}
                     {# CAREFUL, this should only be shown to approvers! #}
+                    {% if access.internal_reason %}
                     <p>The internal notes the person marking this {{ access.frontend.type|lower }} as {{ access.status|lower }} wrote were:</p>
+                {% endif %}
                     <p>{{ access.internal_reason|markdown }}</p>
                     {% if user.is_staff %}
+                        {% if access.internal_comment %}
                         <p>The internal notes for CEDA staff on this {{ access.frontend.type|lower }} were:</p>
                         <p>{{ access.internal_comment|markdown }}</p>
                     {% endif %}
+                    {% endif %}
                 {% endif %}
                 {% if access.frontend.may_apply and access.status == "INCOMPLETE"%}
                     <a class="btn btn-warning btn-sm" href="{{ access.frontend.apply_url }}">Add Info</a>