Skip to content

Conversation

@carlagn
Copy link
Contributor

@carlagn carlagn commented Sep 9, 2025

Fixes #DC-4975

Summary by CodeRabbit

  • Documentation
    • Removed a static redirect from /postgres/introduction/management-api#post-databasesdatabaseidbackupsbackupidrestore to /postgres/introduction/management-api#post-projectsprojectiddatabases; visits to the legacy anchor will no longer auto-redirect.
    • All other static redirects remain unchanged, including those under /postgres/database/api-reference/management-api.
    • Dynamic redirects are unaffected.
    • No changes to site content or navigation.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Sep 9, 2025

Walkthrough

A single static redirect entry was removed from static/_redirects. It previously mapped an old Management API anchor under /postgres/introduction to a different anchor. No other redirects, including dynamic redirects, were changed.

Changes

Cohort / File(s) Summary
Redirects configuration
`static/_redirects`
Removed one redirect mapping from `/postgres/introduction/management-api#post-databasesdatabaseidbackupsbackupidrestore` to `/postgres/introduction/management-api#post-projectsprojectiddatabases`. All other entries unchanged.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Suggested reviewers

  • petradonka
  • mhessdev

Pre-merge checks (3 passed)

✅ Passed checks (3 passed)
Check name Status Explanation
Title Check ✅ Passed The title concisely and accurately reflects the primary change of updating the redirects list by removing a specific redirect and aligns with the main modification in the pull request.
Description Check ✅ Passed The description clearly references the issue being addressed and is related to the pull request’s change set, satisfying the requirement for a relevant description.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between eb89de2 and f8863c8.

📒 Files selected for processing (1)
  • static/_redirects (0 hunks)
💤 Files with no reviewable changes (1)
  • static/_redirects
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (3)
  • GitHub Check: Check internal links
  • GitHub Check: runner / linkspector
  • GitHub Check: Lost Pixel
✨ Finishing Touches
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/DC-4975-update-redirect

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

@github-actions
Copy link
Contributor

github-actions bot commented Sep 9, 2025

Dangerous URL check

No absolute URLs to prisma.io/docs found.
No local URLs found.

@github-actions
Copy link
Contributor

github-actions bot commented Sep 9, 2025

Redirect check

This PR probably requires the following redirects to be added to static/_redirects:

  • This PR does not change any pages in a way that would require a redirect.

@cloudflare-workers-and-pages
Copy link

Deploying docs with  Cloudflare Pages  Cloudflare Pages

Latest commit: f8863c8
Status: ✅  Deploy successful!
Preview URL: https://f5962e19.docs-51g.pages.dev
Branch Preview URL: https://fix-dc-4975-update-redirect.docs-51g.pages.dev

View logs

@carlagn carlagn merged commit 3172159 into main Sep 10, 2025
9 checks passed
@carlagn carlagn deleted the fix/DC-4975-update-redirect branch September 10, 2025 12:00
@coderabbitai coderabbitai bot mentioned this pull request Sep 30, 2025
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