From 45a93bb27330c206db6e5b1d4d34049b82b37456 Mon Sep 17 00:00:00 2001
From: Paul Marillonnet <pmarillonnet@entrouvert.com>
Date: Tue, 20 Oct 2020 16:48:30 +0200
Subject: [PATCH] manage: hide journal app whose icon is missing (#47891)

---
 .../manager/templates/authentic2/manager/homepage.html    | 1 +
 src/authentic2/manager/views.py                           | 8 --------
 tests/test_manager_journal.py                             | 2 +-
 3 files changed, 2 insertions(+), 9 deletions(-)

diff --git a/src/authentic2/manager/templates/authentic2/manager/homepage.html b/src/authentic2/manager/templates/authentic2/manager/homepage.html
index 495c41e35..a9448fc58 100644
--- a/src/authentic2/manager/templates/authentic2/manager/homepage.html
+++ b/src/authentic2/manager/templates/authentic2/manager/homepage.html
@@ -12,6 +12,7 @@
   <ul class="extra-actions-menu">
     <li><a download href="{% url 'a2-manager-site-export' %}">{% trans 'Export Site' %}</a></li>
     <li><a href="{% url 'a2-manager-site-import' %}" rel="popup">{% trans 'Import Site' %}</a></li>
+    <li><a href="{% url 'a2-manager-journal' %}">{% trans 'Journal' %}</a></li>
   </ul>
   </span>
   {% endif %}
diff --git a/src/authentic2/manager/views.py b/src/authentic2/manager/views.py
index bf97d0816..aacdbfffa 100644
--- a/src/authentic2/manager/views.py
+++ b/src/authentic2/manager/views.py
@@ -599,14 +599,6 @@ class HomepageView(TitleMixin, PermissionMixin, MediaMixin, TemplateView):
             'permission': 'authentic2.search_service',
             'slug': 'services',
         },
-        {
-            'class': 'icon-journal',
-            'href': reverse_lazy('a2-manager-journal'),
-            'label': _('Journal'),
-            'order': -1,
-            'permission': 'superuser',
-            'slug': 'journal',
-        },
     ]
 
     def dispatch(self, request, *args, **kwargs):
diff --git a/tests/test_manager_journal.py b/tests/test_manager_journal.py
index d637ccc4c..b2020d76c 100644
--- a/tests/test_manager_journal.py
+++ b/tests/test_manager_journal.py
@@ -233,7 +233,7 @@ def test_global_journal(app, superuser, events):
     # remove event about admin login
     Event.objects.filter(user=superuser).delete()
 
-    response = response.click(href="journal")
+    response = response.click("Journal")
 
     content = extract_journal(response)