From b2a0a3d1e2ecd2a259ffdb7e99873017a15bfa1c Mon Sep 17 00:00:00 2001 From: 50436582a-stack <50436582a@gmail.com> Date: Mon, 15 Jun 2026 01:59:21 +0800 Subject: [PATCH] fix: redirect to /bounties after login instead of /home 404 MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Fixes #329 — Successful GitHub login redirects to /home which returns 404. The `signed_in_path_from_context("personal")` function returns `~p"/home"` but no such route exists in the router, causing a 404 page after successful GitHub OAuth login. This redirects personal users to /bounties instead, which: - Is a real route (BountiesLive, :index) - Is the natural landing page for developers signing up to claim bounties - Was confirmed working by the bug reporter --- lib/algora_web/controllers/user_auth.ex | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/lib/algora_web/controllers/user_auth.ex b/lib/algora_web/controllers/user_auth.ex index e3f00b465..89948287f 100644 --- a/lib/algora_web/controllers/user_auth.ex +++ b/lib/algora_web/controllers/user_auth.ex @@ -229,12 +229,12 @@ defmodule AlgoraWeb.UserAuth do defp maybe_store_return_to(conn), do: conn - def signed_in_path_from_context("personal"), do: ~p"/home" + def signed_in_path_from_context("personal"), do: ~p"/bounties" def signed_in_path_from_context("preview/" <> ctx) do case String.split(ctx, "/") do [_id, repo_owner, repo_name] -> ~p"/go/#{repo_owner}/#{repo_name}" - _ -> ~p"/home" + _ -> ~p"/bounties" end end @@ -343,7 +343,7 @@ defmodule AlgoraWeb.UserAuth do def login_path(email, token, return_to), do: ~p"/callbacks/email/oauth?email=#{email}&token=#{token}&return_to=#{return_to}" - def generate_login_path(email, return_to \\ nil), do: login_path(email, generate_login_code(email), return_to) + def generate_login_path(email, return_to \ nil), do: login_path(email, generate_login_code(email), return_to) def generate_totp do secret = NimbleTOTP.secret()