fix logout form

contributors/views/user_settings.py

@@ -18,35 +18,37 @@ class ChangeTokenView(
 ):
     """Changing user git_hub token page view."""
 
-    template_name = 'user_settings.html'
-    not_auth_msg = _('Please log in with your GitHub')
-    no_permission_msg = (
-        _("You haven't got permission to access this section")
-    )
-    redirect_url = reverse_lazy('contributors:home')
+    template_name = "contributor/user_settings.html"
+    not_auth_msg = _("Please log in with your GitHub")
+    no_permission_msg = _("You haven't got permission to access this section")
+    redirect_url = reverse_lazy("contributors:home")
 
     def get_context_data(self, **kwargs):
         """Add additional context for the settings."""
         context = super().get_context_data(**kwargs)
-        context['user'] = self.request.user
+        context["user"] = self.request.user
         return context
 
     def post(self, request, *args, **kwargs):
         """Handle form submission."""
         form = UserTokenForm(request.POST)
         if form.is_valid():
             user = request.user
-            github_token = form.cleaned_data['github_token']
+            github_token = form.cleaned_data["github_token"]
             user.github_token = github_token
             user.save()
-            messages.success(request, _('GitHub token changed successfully'))
-            return redirect(reverse_lazy(
-                'contributors:user_settings',
-                kwargs={'slug': kwargs.get('slug')},
-            ))
+            messages.success(request, _("GitHub token changed successfully"))
+            return redirect(
+                reverse_lazy(
+                    "contributors:user_settings",
+                    kwargs={"slug": kwargs.get("slug")},
+                )
+            )
 
-        messages.error(request, _('An error occurred. Please try again'))
-        return redirect(reverse_lazy(
-            'contributors:user_settings',
-            kwargs={'slug': kwargs.get('slug')},
-        ))
+        messages.error(request, _("An error occurred. Please try again"))
+        return redirect(
+            reverse_lazy(
+                "contributors:user_settings",
+                kwargs={"slug": kwargs.get("slug")},
+            )
+        )

templates/components/navbar.html

@@ -67,7 +67,10 @@
               {% if user.contributor %}
                 <a class="dropdown-item" href="{% url 'contributors:user_settings' slug=user.contributor.login %}">{% trans "Settings" %}</a>
               {% endif %}
-              <a class="dropdown-item" href="{% url 'logout' %}">{% trans "Log out" %}</a>
+              <form class="dropdown-item" method="post" action="{% url 'logout' %}">
+                {% csrf_token %}
+                <button type="submit">{% trans "Log out" %}</button>
+              </form>
             </div>
           </li>
         {% else %}