Skip to content

Add post-login Terms & Conditions enforcement#353

Draft
briansmiley wants to merge 2 commits intomainfrom
terms
Draft

Add post-login Terms & Conditions enforcement#353
briansmiley wants to merge 2 commits intomainfrom
terms

Conversation

@briansmiley
Copy link
Copy Markdown
Collaborator

Summary

  • REST endpoints (GET /api/tc/status, POST /api/tc/accept) with JWT auth and dev-mode test token support
  • Frontend redirects to /terms page before app loads if T&C not accepted
  • Scrollable T&C content with scroll-to-bottom gate, styled with scoped CSS
  • DB migration adds tc_accepted_at and tc_version columns to account table
  • Vite proxy fix (ws→http) for REST endpoints through dev proxy

Test plan

  • Fresh login → redirected to /terms
  • Scroll to bottom → checkbox enables → accept → app loads
  • Refresh → straight to app (T&C already accepted)
  • Manual nav to /terms when already accepted → redirects to /
  • Manual nav to /terms when not logged in → redirects to /login

🤖 Generated with Claude Code

briansmiley and others added 2 commits February 25, 2026 10:33
REST endpoints (GET /api/tc/status, POST /api/tc/accept) with JWT auth
and dev-mode test token support. Frontend redirects to /terms page before
connecting WebSocket if T&C not yet accepted. Lazy WebSocket connection
pattern ensures no connection on auth pages.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Connect WebSocket before T&C status check so account row exists for the
accept endpoint. Add scoped CSS for readable T&C formatting. Remove
"Changes to Terms" section.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@vercel
Copy link
Copy Markdown

vercel bot commented Feb 26, 2026

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

Project Deployment Actions Updated (UTC)
platform Building Building Preview, Comment Feb 26, 2026 1:09am

Request Review

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