docs icon indicating copy to clipboard operation
docs copied to clipboard

Fix: Add missing redirect for shadow-database docs (Issue #28586)

Open saikumarvasa100-hash opened this issue 1 month ago β€’ 1 comments

The short URL https://pris.ly/d/migrate-shadow was redirecting to the old documentation path /concepts/components/prisma-migrate/shadow-database which resulted in a 404 error.

This PR adds a redirect from the old path to the new documentation location at /docs/orm/prisma-migrate/understanding-prisma-migrate/shadow-database.

Fixes #28586

Summary by CodeRabbit

  • Chores
    • Removed a legacy documentation redirect, so the old documentation URL no longer forwards to the updated page; users should use the current documentation path directly.

✏️ Tip: You can customize this high-level summary in your review settings.

saikumarvasa100-hash avatar Nov 20 '25 16:11 saikumarvasa100-hash

Walkthrough

Removed a redirect mapping from the static redirects file: the legacy path /concepts/components/prisma-client/working-with-json was deleted; no new redirect was added.

Changes

Cohort / File(s) Summary
Redirect configuration
static/_redirects
Removed redirect entry for /concepts/components/prisma-client/working-with-json that pointed to /docs/concepts/components/prisma-client/working-with-advanced-types

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

  • Files to check: static/_redirects for accidental removal of additional lines or formatting issues.

Possibly related PRs

  • prisma/docs#7250 β€” Also modifies static/_redirects by removing redirect entries; likely related change set affecting redirects.

Pre-merge checks

❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Title check ⚠️ Warning The PR title claims to 'Add missing redirect' but the raw summary shows a redirect was removed, not added, creating a contradiction between title and actual changes. Update the title to accurately reflect that a redirect was removed (e.g., 'Fix: Remove outdated redirect for shadow-database docs') or clarify if this is replacing an old redirect with a new one.
βœ… Passed checks (2 passed)
Check name Status Explanation
Description Check βœ… Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage βœ… Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

πŸ“œ Recent review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

πŸ“₯ Commits

Reviewing files that changed from the base of the PR and between 5d76825884430d53d6ba5391d5e1e0638aa86349 and b78019d8ea18e89f1051a3c5715a30cefdc1704e.

πŸ“’ Files selected for processing (1)
  • static/_redirects (0 hunks)
πŸ’€ Files with no reviewable changes (1)
  • static/_redirects

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❀️ Share

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

coderabbitai[bot] avatar Nov 20 '25 16:11 coderabbitai[bot]