Skip to content

DX-2690: update redis screenshots and quickstarts#704

Merged
CahidArda merged 9 commits into
mainfrom
DX-2690
May 21, 2026
Merged

DX-2690: update redis screenshots and quickstarts#704
CahidArda merged 9 commits into
mainfrom
DX-2690

Conversation

@CahidArda
Copy link
Copy Markdown
Contributor

No description provided.

CahidArda added 4 commits May 20, 2026 20:58
Rename connectclient, connectwithupstashredis, upgradedatabase,
metricsandcharts, and addapaymentmethod to kebab-case. Updates
internal links and adds redirects from old slugs.
- Point readers to the Upstash Console in getstarted and add-payment-method
- Tighten upgrade-database; rely on the pricing page for current limits
- Fix Redis.fromEnv() snippet in connect-with-upstash-redis
- Reword metrics-and-charts intro; drop stale "5 days" claim on Current Month
- Renumber duplicate "## 3." in nextjs-app-router as "## 4."
- Drop stray leading space in cloudflareworkers title
@linear
Copy link
Copy Markdown

linear Bot commented May 21, 2026

DX-2690

@mintlify
Copy link
Copy Markdown

mintlify Bot commented May 21, 2026

Preview deployment for your docs. Learn more about Mintlify Previews.

Project Status Preview Updated (UTC)
upstash 🟢 Ready View Preview May 21, 2026, 8:21 AM

💡 Tip: Enable Workflows to automatically generate PRs for you.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the Upstash Redis docs to use new kebab-case page slugs, refreshes several quickstarts/how-to pages, and updates references/screenshots to match the current Console + SDK messaging.

Changes:

  • Renames multiple Redis/Common docs pages (and their inbound links) to kebab-case equivalents, updating references accordingly.
  • Refreshes @upstash/redis TypeScript docs and related connection guides/quickstarts.
  • Adds redirects in docs.json and updates llms index files to reflect the new routes.

Reviewed changes

Copilot reviewed 18 out of 32 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
redis/troubleshooting/max_requests_limit.mdx Updates upgrade link to new upgrade-database slug.
redis/troubleshooting/max_daily_request_limit.mdx Updates upgrade link to new upgrade-database slug.
redis/sdks/ts/overview.mdx Updates SDK overview copy and links (repo + REST API).
redis/quickstarts/nextjs-app-router.mdx Fixes section numbering in the Next.js App Router quickstart.
redis/quickstarts/cloudflareworkers.mdx Fixes frontmatter title spacing.
redis/overall/getstarted.mdx Refreshes Console instructions, updates “Connect your client” links, adds a tip, and updates MCP link text/route.
redis/howto/upgradedatabase.mdx Removes old upgrade page (replaced by upgrade-database).
redis/howto/upgrade-database.mdx Adds new upgrade page content under kebab-case slug.
redis/howto/metrics-and-charts.mdx Refreshes wording and swaps/updates screenshot references.
redis/howto/getstartedawslambda.mdx Updates link to new connect-with-upstash-redis slug.
redis/howto/connect-with-upstash-redis.mdx Updates page title/content to @upstash/redis, adjusts examples, and adds a tip.
redis/howto/connect-client.mdx Refreshes “connect client” wording and @upstash/redis section text.
redis/help/production-checklist.mdx Updates Monitoring/Metrics link to new metrics page slug.
common/help/production-checklist.mdx Updates Monitoring/Metrics link to new metrics page slug.
common/account/addapaymentmethod.mdx Removes old payment method page (replaced by add-payment-method).
common/account/add-payment-method.mdx Adds new payment method page content under kebab-case slug.
llms.txt Updates several referenced doc URLs to new slugs.
llms-full.txt Updates embedded source URLs/content to match new slugs and copy changes.
docs.json Updates navigation slugs and adds redirect mappings for renamed pages.
Comments suppressed due to low confidence (6)

redis/howto/connect-with-upstash-redis.mdx:17

  • The Fastly example link points to github.com/upstash/upstash-redis/... while the page otherwise links to github.com/upstash/redis-js. This mismatch is likely to confuse readers; please update the example URL(s) to be consistent with the repository you’re documenting for @upstash/redis.
    redis/howto/connect-with-upstash-redis.mdx:45
  • The quickstart snippet uses top-level await (const data = await redis.get(...)). This won’t run in CommonJS or older Node/TS setups without top-level await support and the docs don’t state any prerequisites. Consider wrapping the example in an async IIFE or explicitly noting that it must run in an async context / ESM with top-level await enabled.
    redis/howto/connect-with-upstash-redis.mdx:55
  • There are missing spaces in inline code references (e.g., UPSTASH_REDIS_REST_URL` and`UPSTASH_REDIS_REST_TOKEN). Add a space after and for readability and to avoid rendering issues in some Markdown processors.
    redis/howto/connect-client.mdx:25
  • This section says the console shows Endpoint, Port, and Token (and that the token is also the password). Elsewhere (e.g., the redis-cli example in redis/overall/getstarted) the credential is consistently referred to as a password, and Upstash also has a separate REST token. Consider using Password here (and mentioning REST URL/token separately) to avoid implying the REST token is used for TCP clients.
    redis/howto/connect-client.mdx:11
  • Minor grammar: “Simplest way to connect…” reads awkwardly; consider “The simplest way to connect…” for correctness/consistency with the rest of the docs.
    redis/howto/connect-client.mdx:25
  • Missing space in inline code: UPSTASH_REDIS_REST_URL` and`UPSTASH_REDIS_REST_TOKEN. Add a space after and for readability.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread redis/sdks/ts/overview.mdx
Comment thread docs.json
@CahidArda CahidArda merged commit 432307d into main May 21, 2026
2 checks passed
@CahidArda CahidArda deleted the DX-2690 branch May 21, 2026 10:46
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