Skip to content

fix(data-exports): redirect guests to sign in - #5352

Merged
pandemicsyn merged 1 commit into
mainfrom
fix/data-exports-auth-redirect
Aug 18, 2026
Merged

fix(data-exports): redirect guests to sign in#5352
pandemicsyn merged 1 commit into
mainfrom
fix/data-exports-auth-redirect

Conversation

@pandemicsyn

Copy link
Copy Markdown
Contributor

Summary

  • Replace the /data-exports page's unauthenticated 404 with the standard protected-page sign-in redirect.
  • Preserve /data-exports as the callback path so users following export-ready links return to the intended page after authentication.
  • Update page-level regression coverage for both guest redirects and signed-in non-admin access.

Verification

  • Manual browser verification was not performed because no local development stack or PostgreSQL service was available.

Visual Changes

Before After
Unauthenticated visits to /data-exports render a 404 page. Unauthenticated visits redirect to sign-in and return to /data-exports after authentication.

Reviewer Notes

  • This uses the same getUserFromAuthOrRedirect pattern as other protected app pages and does not change authorization for signed-in users.
  • The focused Jest file was discovered, but repository-wide test setup could not connect to the unavailable local PostgreSQL service before assertions ran.

@kilo-code-bot

kilo-code-bot Bot commented Aug 18, 2026

Copy link
Copy Markdown
Contributor

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Files Reviewed (2 files)
  • apps/web/src/app/(app)/data-exports/page.tsx
  • apps/web/src/app/(app)/data-exports/page.test.ts

Reviewed by grok-4.6 · Input: 43.7K · Output: 3K · Cached: 208.9K

Review guidance: REVIEW.md from base branch main

@pandemicsyn
pandemicsyn enabled auto-merge (squash) August 18, 2026 17:49
@pandemicsyn
pandemicsyn merged commit aafb20d into main Aug 18, 2026
13 checks passed
@pandemicsyn
pandemicsyn deleted the fix/data-exports-auth-redirect branch August 18, 2026 17:57
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.

2 participants