Remove Success Messages

risks/views.py

@@ -375,7 +375,7 @@ def notification_mark_read(request, pk):
     notif = get_object_or_404(Notification, pk=pk, user=request.user)
     notif.read = True
     notif.save(update_fields=["read"])
-    messages.success(request, _("Notification marked as read."))
+    #messages.success(request, _("Notification marked as read."))
     return redirect(request.META.get("HTTP_REFERER") or "risks:notifications")
 
 
@@ -385,7 +385,7 @@ def notification_mark_all_read(request):
     if request.method != "POST":
         return HttpResponseForbidden()
     Notification.objects.filter(user=request.user, read=False).update(read=True)
-    messages.success(request, _("All notifications marked as read."))
+    #messages.success(request, _("All notifications marked as read."))
     return redirect("risks:notifications")
 
 

templates/risks/item_risk.html

@@ -21,11 +21,13 @@
     <a href="{% url 'admin:risks_risk_delete' risk.pk %}" class="button is-small is-danger" title="{% trans 'Delete Risk' %}">
       <span class="icon"><i class="fas fa-trash"></i></span>&nbsp;&nbsp;{% trans 'Delete Risk' %}
     </a>
+    {% if risk.status != 'closed' %}
     <a href="{% url 'risks:mark_risk_reviewed' risk.pk %}" 
       class="button is-small is-success" 
       title="{% trans 'Mark Risk as reviewed' %}">
       <span class="icon"><i class="fas fa-check"></i></span>&nbsp;&nbsp;{% trans 'Mark Risk as reviewed' %}
     </a>
+    {% endif %}
   </div>
 </div>
 
@@ -48,7 +50,20 @@
                 <span class="has-text-grey">{% trans "Not yet assigned" %}</span>
               {% endif %}
             </p>
-            <p><strong>{% trans "Status" %}:</strong> {{ risk.get_status_display }}</p>
+            <p>
+              <strong>{% trans "Status" %}:</strong>
+              <strong  class="tag 
+                {% if risk.status == 'open' %}
+                  is-info
+                {% elif risk.status == 'review_required' %}
+                  is-warning
+                {% elif risk.status == 'closed' %}
+                  is-success
+                {% elif risk.status == 'in_progress' %}
+                  is-link is-light
+                {% endif %}
+              ">{{ risk.get_status_display }}
+            </strong>
           </div>
           <div class="column is-half">
             <p><strong>{% trans "Risk owner" %}:</strong> {{ risk.owner|user_display|default:"-" }}</p>