Skip to content

Revert "Login Setup"#9

Merged
orimcoding merged 1 commit intomainfrom
revert-8-login-setup
Apr 22, 2026
Merged

Revert "Login Setup"#9
orimcoding merged 1 commit intomainfrom
revert-8-login-setup

Conversation

@orimcoding
Copy link
Copy Markdown
Owner

@orimcoding orimcoding commented Apr 22, 2026

Reverts #8

Summary by CodeRabbit

Release Notes

  • Changes
    • Removed login page and login form components
    • Removed user dashboard page and associated features
    • Removed logout API endpoint
    • Redesigned home page with new static landing interface, removing authentication-driven redirects
    • Simplified authentication middleware logic

This reverts commit e62b105.
@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 22, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
swale Ready Ready Preview, Comment Apr 22, 2026 10:47pm

@supabase
Copy link
Copy Markdown

supabase Bot commented Apr 22, 2026

This pull request has been ignored for the connected project isbbtnzupwefmhzojltw because there are no changes detected in supabase directory. You can change this behaviour in Project Integrations Settings ↗︎.


Preview Branches by Supabase.
Learn more about Supabase Branching ↗︎.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Apr 22, 2026

Caution

Review failed

The pull request is closed.

Note

.coderabbit.yaml has unrecognized properties

CodeRabbit is using all valid settings from your configuration. Unrecognized properties (listed below) have been ignored and may indicate typos or deprecated fields that can be removed.

⚠️ Parsing warnings (1)
Validation error: Unrecognized key(s) in object: 'version'
⚙️ Configuration instructions
  • Please see the configuration documentation for more information.
  • You can also validate your configuration using the online YAML validator.
  • If your editor has YAML language server enabled, you can add the path at the top of this file to enable auto-completion and validation: # yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json
ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: 0178879a-d2c8-4be1-a204-95fd104db260

📥 Commits

Reviewing files that changed from the base of the PR and between e62b105 and ca88a0c.

📒 Files selected for processing (10)
  • src/app/api/auth/logout/route.ts
  • src/app/auth/layout.tsx
  • src/app/auth/login/page.tsx
  • src/app/dashboard/page.tsx
  • src/app/page.tsx
  • src/components/auth/LoginForm.tsx
  • src/components/auth/LoginPageUI.tsx
  • src/lib/supabase/client.ts
  • src/lib/supabase/server.ts
  • src/middleware.ts

📝 Walkthrough

Walkthrough

This PR removes the entire authentication system, including login/logout endpoints, dashboard page, and auth-related UI components. It replaces the home page with a static landing page and simplifies middleware by removing auth-based redirects. Environment variable validation is replaced with non-null assertions.

Changes

Cohort / File(s) Summary
Authentication API Routes
src/app/api/auth/logout/route.ts
Removed POST and GET handlers that called supabase.auth.signOut() and returned JSON responses or redirects.
Authentication Pages & Layout
src/app/auth/layout.tsx, src/app/auth/login/page.tsx
Deleted AuthLayout wrapper and LoginPage module entirely.
Authentication Components
src/components/auth/LoginForm.tsx, src/components/auth/LoginPageUI.tsx
Removed login form component handling email/password submission and the UI wrapper combining branding and login form.
Core Pages
src/app/page.tsx, src/app/dashboard/page.tsx
Replaced home page's client-side auth checks and redirects with static landing page template; removed entire dashboard page that displayed authenticated user email and placeholder cards.
Supabase Integration
src/lib/supabase/client.ts, src/lib/supabase/server.ts
Replaced environment variable validation with non-null assertions; removed error throwing on missing Supabase configuration.
Middleware
src/middleware.ts
Removed auth-based user inspection, protected route redirects (/dashboard), and authenticated user redirects from home/auth pages. Simplified to basic user check without conditional logic.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

  • PR #5: Modifies Supabase integration in middleware and client/server utilities alongside auth API route handlers that this PR removes.
  • PR #8: Introduces the auth routes, layout, login components, and middleware logic that this PR comprehensively removes.

Poem

🐰 A hop through the code, what do we find?
Auth flows and dashboards left far behind!
Static pages gleam, middleware is lean,
The simplest login system ever seen!

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch revert-8-login-setup

Comment @coderabbitai help to get the list of available commands and usage tips.

@orimcoding orimcoding merged commit ab52c38 into main Apr 22, 2026
8 of 9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant