Skip to content

Remove /login route in favour of / landing page - #112

Merged
devarshishimpi merged 3 commits into
devfrom
fix/dup-login-route
Sep 18, 2026
Merged

devarshishimpi merged 3 commits into
devfrom
fix/dup-login-route

Conversation

@devarshishimpi

Copy link
Copy Markdown
Owner

Description

The /login route was redundant — the / landing page already contains
the full sign-in UI (GitHub OAuth button, theme toggle, terms/privacy links).

Changes

  • Remove /login route and LoginPage lazy import from routes.tsx
  • Delete pages/login.tsx (dead code)
  • Update 401 unauthorized redirects in api.ts from /login/
  • Update post-logout redirect in account-menu.tsx from /login/
  • Remove /login from run_worker_first in wrangler.jsonc

Closes #110

@devarshishimpi
devarshishimpi marked this pull request as ready for review September 18, 2026 18:10

@codra-app-personal codra-app-personal Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Codra Review

Nothing to flag. Reviewed 4 files (53 changed lines) and found no issues worth raising.

Reviewed commit: 53088234e4

ℹ️ About Codra in GitHub

Your team has set up Codra to review pull requests in this repo. Reviews are triggered when you:

  • Open a pull request for review
  • Mark a draft as ready

Every review posts a summary here. A clean pass also gets a 👍 on the pull request itself.

@codra-app-personal codra-app-personal Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Codra Review

Nothing to flag. Reviewed 5 files (75 changed lines) and found no issues worth raising.

Reviewed commit: 6642d47067

ℹ️ About Codra in GitHub

Your team has set up Codra to review pull requests in this repo. Reviews are triggered when you:

  • Open a pull request for review
  • Mark a draft as ready

Every review posts a summary here. A clean pass also gets a 👍 on the pull request itself.

@devarshishimpi devarshishimpi linked an issue Sep 18, 2026 that may be closed by this pull request

@codra-app-personal codra-app-personal Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Codra Review

Nothing to flag. Reviewed 6 files (83 changed lines) and found no issues worth raising.

Reviewed commit: a660f86cc5

ℹ️ About Codra in GitHub

Your team has set up Codra to review pull requests in this repo. Reviews are triggered when you:

  • Open a pull request for review
  • Mark a draft as ready

Every review posts a summary here. A clean pass also gets a 👍 on the pull request itself.

@devarshishimpi
devarshishimpi merged commit 6a2f2d6 into dev Sep 18, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Redesign Login/Signup Page (Modern UI with GitHub OAuth)

1 participant