Remove /login route in favour of / landing page - #112
Conversation
There was a problem hiding this comment.
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.
There was a problem hiding this comment.
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.
There was a problem hiding this comment.
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.
Description
The
/loginroute was redundant — the/landing page already containsthe full sign-in UI (GitHub OAuth button, theme toggle, terms/privacy links).
Changes
/loginroute andLoginPagelazy import fromroutes.tsxpages/login.tsx(dead code)api.tsfrom/login→/account-menu.tsxfrom/login→//loginfromrun_worker_firstinwrangler.jsoncCloses #110