Fix bug in plug pipelines

This prevented the MethodOverride plug from working, which allows us to make links that have a `DELETE` method for example.

Also update the logout button to use delete.

lib/bike_brigade_web/components/layouts.ex

@@ -77,7 +77,7 @@ defmodule BikeBrigadeWeb.Layouts do
           My Profile
         </.sidebar_link>
 
-        <.sidebar_link selected={@current_page == :logout} href={~p"/logout"} method="post">
+        <.sidebar_link selected={@current_page == :logout} href={~p"/logout"} method="delete">
           <:icon>
             <Heroicons.arrow_left_on_rectangle solid />
           </:icon>

lib/bike_brigade_web/endpoint.ex

@@ -48,6 +48,11 @@ defmodule BikeBrigadeWeb.Endpoint do
   plug Plug.RequestId
   plug Plug.Telemetry, event_prefix: [:phoenix, :endpoint]
 
+  plug Plug.Parsers,
+    parsers: [:urlencoded, :multipart, :json],
+    pass: ["*/*"],
+    json_decoder: Phoenix.json_library()
+
   plug Plug.MethodOverride
   plug Plug.Head
   plug Plug.Session, @session_options

lib/bike_brigade_web/router.ex

@@ -27,15 +27,7 @@ defmodule BikeBrigadeWeb.Router do
     plug :get_user_from_session
   end
 
-  pipeline :parse_request do
-    plug Plug.Parsers,
-      parsers: [:urlencoded, :multipart, :json],
-      pass: ["*/*"],
-      json_decoder: Phoenix.json_library()
-  end
-
   pipeline :browser do
-    plug :parse_request
     plug :accepts, ["html"]
     plug :sessions
     plug :fetch_live_flash
@@ -46,7 +38,6 @@ defmodule BikeBrigadeWeb.Router do
   end
 
   pipeline :api do
-    plug :parse_request
     plug :accepts, ["json"]
   end
 
@@ -110,7 +101,7 @@ defmodule BikeBrigadeWeb.Router do
       live "/leaderboard", StatsLive.Leaderboard, :show
     end
 
-    post "/logout", Authentication, :logout
+    delete "/logout", Authentication, :logout
   end
 
   scope "/", BikeBrigadeWeb do