Merge pull request #1230 from thunderstore-io/nav-to-team-settings-if-no-team

Fix user nav linking to unowned pages

Author: Roffenlund

django/thunderstore/frontend/templates/base.html

@@ -88,7 +88,11 @@ <h6 class="title">Popular communities</h6>
                 {% dynamic_html "nav_bar_right_nav" %}
                 {% if request.user.is_authenticated %}
                     <li class="nav-item">
-                        <a href="{% community_url 'packages.list_by_owner' owner=team %}" class="nav-link text-dark">{{ request.user.username }}</a>
+                        {% if team %}
+                            <a title="View packages uploaded by your first team" href="{% community_url 'packages.list_by_owner' owner=team %}" class="nav-link text-dark">{{ request.user.username }}</a>
+                        {% else %}
+                            <a title="Open team settings" href="{% url "settings.teams" %}" class="nav-link text-dark">{{ request.user.username }}</a>
+                        {% endif %}
                     </li>
                     <li class="nav-item">
                         <a href="{% url "settings.linked-accounts" %}" class="nav-link">Settings</a>

django/thunderstore/repository/context_processors.py

@@ -1,8 +1,7 @@
 def team(request):
     if not (hasattr(request, "user") and request.user.is_authenticated):
         return {}
-    name = request.user.username
     membership = request.user.teams.filter(team__is_active=True).first()
     if membership:
-        name = membership.team.name
-    return {"team": name}
+        return {"team": membership.team.name}
+    return {}