Merge pull request #451 from WikipediaLibrary/kgraessle/add-warning-banners-temporarily

Create banners to indicate wrong data on Wikilink

Author: suecarmol

extlinks/organisations/templates/organisations/organisation_detail.html

@@ -16,6 +16,15 @@
       </ul>
     </nav>
     <nav id="content" class="charts-body">
+      {% if messages %}
+        <div class="messages">
+            {% for message in messages %}
+                <h3>
+                    {{ message }}
+                </h3>
+            {% endfor %}
+        </div>
+      {% endif %}
       <div class="row">
         <div class="col-6">
           <h1>{{ object }}</h1>

extlinks/organisations/views.py

@@ -2,6 +2,7 @@
 import json
 import re
 
+from django.contrib import messages
 from django.db.models import Count, Sum, Q, Prefetch, CharField
 from django.db.models.functions import Cast
 from django.http import JsonResponse
@@ -47,6 +48,12 @@ def get_queryset(self, **kwargs):
     # This is almost, but not exactly, the same as the program view.
     # As such, most context gathering is split out to a helper.
     def get_context_data(self, **kwargs):
+        messages.warning(
+            self.request,
+            "We have modified where Wikilink obtains its data from. Since some of this work is "
+            "still in flight, the data shown in Wikilink is currently erroneous. ",
+            fail_silently=True,
+        )
         context = super(OrganisationDetailView, self).get_context_data(**kwargs)
         form = self.form_class(self.request.GET)
         context["form"] = form
@@ -76,7 +83,9 @@ def get_context_data(self, **kwargs):
             context["collections"][collection_key] = {}
             context["collections"][collection_key]["object"] = collection
             context["collections"][collection_key]["collection_id"] = collection.pk
-            context["collections"][collection_key]["urls"] = collection.get_url_patterns()
+            context["collections"][collection_key][
+                "urls"
+            ] = collection.get_url_patterns()
 
             context["collections"][collection_key] = (
                 self._build_collection_context_dictionary(

extlinks/programs/templates/programs/program_detail.html

@@ -13,6 +13,15 @@
       </ul>
     </nav>
     <nav id="content" class="charts-body">
+      {% if messages %}
+        <div class="messages">
+            {% for message in messages %}
+                <h3>
+                    {{ message }}
+                </h3>
+            {% endfor %}
+        </div>
+    {% endif %}
       <div class="row">
         <div class="col-6">
           <h1>{{ object }}</h1>

extlinks/programs/views.py

@@ -1,6 +1,7 @@
 from datetime import date, timedelta
 import json
 
+from django.contrib import messages
 from django.db.models import Sum, Count, Q
 from django.http import JsonResponse
 from django.views.generic import ListView, DetailView
@@ -42,6 +43,12 @@ def get_queryset(self, **kwargs):
 
     def get_context_data(self, **kwargs):
         context = super(ProgramDetailView, self).get_context_data(**kwargs)
+        messages.warning(
+            self.request,
+            "We have modified where Wikilink obtains its data from. Since some of this work is "
+            "still in flight, the data shown in Wikilink is currently erroneous. ",
+            fail_silently=True,
+        )
         this_program_organisations = self.object.organisation_set.all()
         context["organisations"] = this_program_organisations
         context["program_id"] = self.object.pk

static/css/local.css

@@ -123,3 +123,13 @@ hr {
     border-top-color: #9a9a9a;
     border-top-width: 4px;
 }
+
+.messages{
+    display: flex;
+    justify-content: center;
+    margin: 2rem;
+    background-color: #fdf2d5;
+    border-style: solid;
+    border-width: 0.25em;
+    border-color: #fbe2a2;
+}