feat: sponsor directory and asset browser guide documentation

JacobCoffee · claude · JacobCoffee · commit de3fab82ad0c · 2026-03-24T23:40:14.000-05:00
Adds a Sponsors page (More &gt; Sponsors) to browse and search all sponsor
companies with contact count, sponsorship count, location, website, and
quick links to edit or create a sponsorship via the Composer.

Also adds the Assets section to the guide documenting the grouped asset
browser, filters, and Send Reminder feature. Updates sponsor edit to
redirect back to the sponsor directory.

Includes 4 tests.

Co-Authored-By: Claude Opus 4.6 (1M context) &lt;noreply@anthropic.com&gt;
diff --git a/apps/sponsors/manage/tests.py b/apps/sponsors/manage/tests.py
@@ -2533,3 +2533,43 @@ def test_shows_active_sponsorship_assets(self):
     def test_nav_has_assets_link(self):
         response = self.client.get(reverse("manage_dashboard"))
         self.assertContains(response, "Assets")
+
+
+class SponsorListViewTests(SponsorManageTestBase):
+    """Test sponsor directory view."""
+
+    @classmethod
+    def setUpTestData(cls):
+        super().setUpTestData()
+        cls.sponsor = Sponsor.objects.create(name="Acme Corp", city="Portland", country="US")
+        cls.sponsor2 = Sponsor.objects.create(name="Beta Inc", city="London", country="GB")
+
+    def setUp(self):
+        super().setUp()
+        self.client.login(username="staff", password="pass")
+
+    def test_list_loads(self):
+        response = self.client.get(reverse("manage_sponsors"))
+        self.assertEqual(response.status_code, 200)
+        self.assertContains(response, "Acme Corp")
+        self.assertContains(response, "Beta Inc")
+
+    def test_search(self):
+        response = self.client.get(reverse("manage_sponsors") + "?search=Acme")
+        self.assertContains(response, "Acme Corp")
+        self.assertNotContains(response, "Beta Inc")
+
+    def test_shows_sponsorship_count(self):
+        Sponsorship.objects.create(
+            sponsor=self.sponsor,
+            submited_by=self.staff_user,
+            package=self.package,
+            year=self.year,
+            status=Sponsorship.APPLIED,
+        )
+        response = self.client.get(reverse("manage_sponsors"))
+        self.assertEqual(response.status_code, 200)
+
+    def test_nav_has_sponsors_link(self):
+        response = self.client.get(reverse("manage_dashboard"))
+        self.assertContains(response, "Sponsors")
diff --git a/apps/sponsors/manage/urls.py b/apps/sponsors/manage/urls.py
@@ -96,7 +96,8 @@
         views.AssetExportView.as_view(),
         name="manage_sponsorship_export_assets",
     ),
-    # Sponsor (company) create/edit
+    # Sponsor directory + create/edit
+    path("sponsors/", views.SponsorListView.as_view(), name="manage_sponsors"),
     path("sponsors/new/", views.SponsorCreateView.as_view(), name="manage_sponsor_create"),
     path("sponsors/<int:pk>/edit/", views.SponsorEditView.as_view(), name="manage_sponsor_edit"),
     # Sponsor contacts
diff --git a/apps/sponsors/manage/views.py b/apps/sponsors/manage/views.py
@@ -555,6 +555,36 @@ def get_context_data(self, **kwargs):
         return context
 
 
+# ── Sponsor Directory ─────────────────────────────────────────────────
+
+
+class SponsorListView(SponsorshipAdminRequiredMixin, ListView):
+    """Browse and search all sponsors."""
+
+    template_name = "sponsors/manage/sponsor_list.html"
+    context_object_name = "sponsors"
+    paginate_by = 50
+
+    def get_queryset(self):
+        """Return sponsors filtered by search, annotated with sponsorship count."""
+        from django.db.models import Count
+
+        qs = Sponsor.objects.annotate(
+            sponsorship_count=Count("sponsorship"),
+            contact_count=Count("contacts"),
+        ).order_by("name")
+        self.filter_search = self.request.GET.get("search", "")
+        if self.filter_search:
+            qs = qs.filter(name__icontains=self.filter_search)
+        return qs
+
+    def get_context_data(self, **kwargs):
+        """Return context with search term."""
+        context = super().get_context_data(**kwargs)
+        context["filter_search"] = self.filter_search
+        return context
+
+
 class PackageListView(SponsorshipAdminRequiredMixin, ListView):
     """List sponsorship packages grouped by year."""
 
@@ -1127,7 +1157,7 @@ def get_success_url(self):
         sp_pk = self.request.POST.get("from_sponsorship") or self.request.GET.get("from_sponsorship")
         if sp_pk:
             return reverse("manage_sponsorship_detail", args=[sp_pk])
-        return reverse("manage_sponsorships")
+        return reverse("manage_sponsors")
 
 
 # ── Benefit management on sponsorships ────────────────────────────────
diff --git a/apps/sponsors/templates/sponsors/manage/_base.html b/apps/sponsors/templates/sponsors/manage/_base.html
@@ -679,6 +679,7 @@
             <div class="manage-nav-more" style="position:relative;">
                 <a href="#" class="{% if active_tab == 'clone' or active_tab == 'guide' %}active{% endif %}" onclick="event.preventDefault();this.parentElement.classList.toggle('open');">More &#9662;</a>
                 <div class="manage-nav-dropdown">
+                    <a href="{% url 'manage_sponsors' %}">Sponsors</a>
                     <a href="{% url 'manage_legal_clauses' %}">Legal Clauses</a>
                     <a href="{% url 'manage_assets' %}">Assets</a>
                     <a href="{% url 'manage_clone_year' %}">Clone Year</a>
diff --git a/apps/sponsors/templates/sponsors/manage/guide.html b/apps/sponsors/templates/sponsors/manage/guide.html
@@ -23,6 +23,7 @@ <h2 style="font-size:22px;font-weight:700;color:#1a1a2e;margin:0 0 8px;">Sponsor
 <a href="#notifications" style="color:#3776ab;text-decoration:none;">Notifications</a><br>
 <a href="#sponsors-contacts" style="color:#3776ab;text-decoration:none;">Sponsors &amp; Contacts</a><br>
 <a href="#renewals" style="color:#3776ab;text-decoration:none;">Renewals &amp; Expiry</a><br>
+<a href="#assets" style="color:#3776ab;text-decoration:none;">Assets</a><br>
 <a href="#bulk" style="color:#3776ab;text-decoration:none;">Bulk Actions &amp; Export</a><br>
 <a href="#admin-only" style="color:#3776ab;text-decoration:none;">What's Still in Django Admin</a><br>
 </div>
@@ -266,7 +267,7 @@ <h3 style="font-size:16px;font-weight:700;color:#3776ab;border-bottom:2px solid
 <h3 style="font-size:16px;font-weight:700;color:#3776ab;border-bottom:2px solid #3776ab;padding-bottom:6px;">Sponsors &amp; Contacts</h3>
 
 <p style="font-size:13px;line-height:1.8;color:#444;">
-A sponsor is the company entity. Create new sponsors from the Composer (step 1) or from the <a href="{% url 'manage_sponsor_create' %}" style="color:#3776ab;">direct create page</a>. Edit a sponsor's company info (name, description, website, address, phone) from their sponsorship detail page by clicking "Edit Sponsor."
+A sponsor is the company entity. Browse all sponsors from <a href="{% url 'manage_sponsors' %}" style="color:#3776ab;">More &rarr; Sponsors</a> &mdash; search by name, see contact and sponsorship counts, and jump to edit or create a new sponsorship. You can also create new sponsors from the Composer (step 1) or from the <a href="{% url 'manage_sponsor_create' %}" style="color:#3776ab;">direct create page</a>. Edit a sponsor's company info (name, description, website, address, phone) from their sponsorship detail page by clicking "Edit Sponsor."
 </p>
 
 <p style="font-size:13px;line-height:1.8;color:#444;">
@@ -311,6 +312,25 @@ <h3 style="font-size:16px;font-weight:700;color:#3776ab;border-bottom:2px solid
 </div>
 </div>
 
+<!-- Assets -->
+<div class="manage-section" id="assets">
+<h3 style="font-size:16px;font-weight:700;color:#3776ab;border-bottom:2px solid #3776ab;padding-bottom:6px;">Assets</h3>
+
+<p style="font-size:13px;line-height:1.8;color:#444;">
+The <a href="{% url 'manage_assets' %}" style="color:#3776ab;">Asset Browser</a> (under More) shows all sponsor-submitted assets grouped by company. Each company section has a submitted/total badge:
+</p>
+
+<ul style="font-size:13px;line-height:2;color:#444;padding-left:20px;">
+<li><span class="tag tag-green" style="font-size:10px;">green</span> &mdash; all assets submitted</li>
+<li><span class="tag tag-gold" style="font-size:10px;">orange</span> &mdash; partially submitted</li>
+<li><span class="tag tag-red" style="font-size:10px;">red</span> &mdash; nothing submitted</li>
+</ul>
+
+<p style="font-size:13px;line-height:1.8;color:#444;">
+Filter by asset type (text, image, file, response), related object (sponsor vs sponsorship), submission status, or search by internal name. Assets from expired or rejected sponsorships are automatically hidden. Companies with missing assets show a <strong>"Send Reminder"</strong> button that links to the notification page for that sponsorship.
+</p>
+</div>
+
 <!-- Bulk Actions & Export -->
 <div class="manage-section" id="bulk">
 <h3 style="font-size:16px;font-weight:700;color:#3776ab;border-bottom:2px solid #3776ab;padding-bottom:6px;">Bulk Actions &amp; Export</h3>
diff --git a/apps/sponsors/templates/sponsors/manage/sponsor_list.html b/apps/sponsors/templates/sponsors/manage/sponsor_list.html
@@ -0,0 +1,111 @@
+{% extends "sponsors/manage/_base.html" %}
+
+{% block page_title %}Sponsors | Sponsor Management{% endblock %}
+
+{% with active_tab="more" %}
+
+{% block manage_breadcrumbs %}
+<div class="manage-crumbs">
+    <a href="{% url 'manage_dashboard' %}">Dashboard</a>
+    <span class="sep">/</span>
+    <span class="current">Sponsors</span>
+</div>
+{% endblock %}
+
+{% block topbar_actions %}
+    <a href="{% url 'manage_sponsor_create' %}" class="btn btn-sm btn-primary">+ New Sponsor</a>
+{% endblock %}
+
+{% block manage_content %}
+    <!-- Search bar -->
+    <div class="manage-filter-bar" style="padding:12px 20px;">
+        <form method="get" style="display:flex;gap:10px;align-items:flex-end;flex-wrap:wrap;">
+            <div class="filter-field">
+                <input type="text" name="search" placeholder="Search by name..." value="{{ filter_search }}" style="width:240px;" autofocus>
+            </div>
+            <div class="filter-actions">
+                <button type="submit" class="btn btn-sm btn-secondary">Search</button>
+                {% if filter_search %}
+                <a href="{% url 'manage_sponsors' %}" class="btn btn-sm btn-secondary">Clear</a>
+                {% endif %}
+            </div>
+            <div style="margin-left:auto;font-size:12px;color:#777;">
+                {{ page_obj.paginator.count }} sponsor{{ page_obj.paginator.count|pluralize }}
+            </div>
+        </form>
+    </div>
+
+    {% if sponsors %}
+    <table class="manage-table">
+        <thead>
+            <tr>
+                <th>Company</th>
+                <th>Location</th>
+                <th style="text-align:center;">Contacts</th>
+                <th style="text-align:center;">Sponsorships</th>
+                <th>Website</th>
+                <th style="width:140px;text-align:right;">Actions</th>
+            </tr>
+        </thead>
+        <tbody>
+            {% for sponsor in sponsors %}
+            <tr>
+                <td>
+                    <a href="{% url 'manage_sponsor_edit' sponsor.pk %}" style="font-weight:600;color:#1a1a2e;text-decoration:none;">
+                        {{ sponsor.name }}
+                    </a>
+                </td>
+                <td style="font-size:12px;color:#777;">
+                    {% if sponsor.city %}{{ sponsor.city }}{% endif %}{% if sponsor.city and sponsor.country %}, {% endif %}{% if sponsor.country %}{{ sponsor.country }}{% endif %}
+                </td>
+                <td style="text-align:center;">
+                    {% if sponsor.contact_count %}
+                    <span class="tag tag-blue" style="font-size:10px;">{{ sponsor.contact_count }}</span>
+                    {% else %}
+                    <span style="color:#ccc;">&mdash;</span>
+                    {% endif %}
+                </td>
+                <td style="text-align:center;">
+                    {% if sponsor.sponsorship_count %}
+                    <a href="{% url 'manage_sponsorships' %}?search={{ sponsor.name|urlencode }}" class="tag tag-green" style="font-size:10px;text-decoration:none;">{{ sponsor.sponsorship_count }}</a>
+                    {% else %}
+                    <span style="color:#ccc;">&mdash;</span>
+                    {% endif %}
+                </td>
+                <td style="font-size:12px;">
+                    {% if sponsor.landing_page_url %}
+                    <a href="{{ sponsor.landing_page_url }}" target="_blank" style="color:#3776ab;text-decoration:none;">{{ sponsor.landing_page_url|truncatechars:30 }}</a>
+                    {% else %}
+                    <span style="color:#ccc;">&mdash;</span>
+                    {% endif %}
+                </td>
+                <td style="text-align:right;">
+                    <a href="{% url 'manage_sponsor_edit' sponsor.pk %}" style="font-size:12px;color:#3776ab;text-decoration:none;margin-right:8px;">Edit</a>
+                    <a href="{% url 'manage_composer' %}?new=1&sponsor_id={{ sponsor.pk }}" style="font-size:12px;color:#3776ab;text-decoration:none;">+ Sponsorship</a>
+                </td>
+            </tr>
+            {% endfor %}
+        </tbody>
+    </table>
+
+    {% if is_paginated %}
+    <div style="display:flex;justify-content:center;gap:4px;margin-top:20px;">
+        {% if page_obj.has_previous %}
+        <a href="?page={{ page_obj.previous_page_number }}{% if filter_search %}&search={{ filter_search }}{% endif %}" class="btn btn-sm btn-secondary">&larr; Prev</a>
+        {% endif %}
+        <span style="padding:6px 12px;font-size:13px;color:#555;">{{ page_obj.number }} / {{ page_obj.paginator.num_pages }}</span>
+        {% if page_obj.has_next %}
+        <a href="?page={{ page_obj.next_page_number }}{% if filter_search %}&search={{ filter_search }}{% endif %}" class="btn btn-sm btn-secondary">Next &rarr;</a>
+        {% endif %}
+    </div>
+    {% endif %}
+
+    {% else %}
+    <div class="empty-state">
+        <div class="empty-icon">&#127970;</div>
+        <p>{% if filter_search %}No sponsors match "{{ filter_search }}"{% else %}No sponsors found{% endif %}</p>
+    </div>
+    {% endif %}
+{% endblock %}
+
+{% endwith %}
