Skip to content

refactor: make kv redis optional#326

Merged
drankou merged 3 commits into
mainfrom
make-redis-optional-for-local-development-eng-3976
May 22, 2026
Merged

refactor: make kv redis optional#326
drankou merged 3 commits into
mainfrom
make-redis-optional-for-local-development-eng-3976

Conversation

@drankou
Copy link
Copy Markdown
Contributor

@drankou drankou commented May 14, 2026

@vercel/kv redis is now optional for the dashboard. The app boots and runs core auth + dashboard flows without KV_REST_API_URL / KV_REST_API_TOKEN configured.

@cla-bot cla-bot Bot added the cla-signed label May 14, 2026
@linear-code
Copy link
Copy Markdown

linear-code Bot commented May 14, 2026

ENG-3976

@vercel
Copy link
Copy Markdown

vercel Bot commented May 14, 2026

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

Project Deployment Actions Updated (UTC)
web Ready Ready Preview, Comment May 22, 2026 1:51pm
web-juliett Ready Ready Preview, Comment May 22, 2026 1:51pm

Request Review

@drankou drankou marked this pull request as ready for review May 14, 2026 14:41
@drankou drankou requested a review from ben-fornefeld as a code owner May 14, 2026 14:41
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 71068a7bcd

ℹ️ About Codex in GitHub

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

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread src/lib/env.ts
Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

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

Stale comment

Comment thread src/core/server/functions/auth/validate-email.ts
Comment thread scripts/check-app-env.ts
Copy link
Copy Markdown
Member

@ben-fornefeld ben-fornefeld left a comment

Choose a reason for hiding this comment

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

lgtm, please check/resolve the bot comments

@drankou drankou force-pushed the make-redis-optional-for-local-development-eng-3976 branch from e80086f to 08cd97b Compare May 22, 2026 13:48
@drankou drankou merged commit d6b8c08 into main May 22, 2026
14 checks passed
@drankou drankou deleted the make-redis-optional-for-local-development-eng-3976 branch May 22, 2026 14:18
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.

2 participants