Skip to content

docs(redirects): add /docs/sdk/python/sdk-v4 -> observability overview (#2917)#2967

Open
mvanhorn wants to merge 1 commit into
langfuse:mainfrom
mvanhorn:docs/2917-add-sdk-v4-redirect
Open

docs(redirects): add /docs/sdk/python/sdk-v4 -> observability overview (#2917)#2967
mvanhorn wants to merge 1 commit into
langfuse:mainfrom
mvanhorn:docs/2917-add-sdk-v4-redirect

Conversation

@mvanhorn
Copy link
Copy Markdown

@mvanhorn mvanhorn commented May 19, 2026

Closes #2917.

Adds a redirect for /docs/sdk/python/sdk-v4 to /docs/observability/sdk/overview, matching the existing sibling entry for /docs/sdk/python/sdk-v3 in lib/redirects.js. The legacy URL now resolves to the same overview page instead of 404'ing.

Change Type

  • Documentation (redirect entry)

Verification

  • Diff is the single-line entry inserted alphabetically next to the existing sdk-v3 redirect, no surrounding edits.
  • Same redirect target as sdk-v3, sdk/python/example, and sdk/python/low-level-sdk — consistent with the rest of the python SDK section in mainDocsReorder202507.

Greptile Summary

This PR adds a single redirect entry mapping the legacy /docs/sdk/python/sdk-v4 URL to /docs/observability/sdk/overview, fixing a 404 for that path.

  • The redirect is inserted alphabetically alongside its sibling entries (sdk-v3, python/example, python/low-level-sdk) in the mainDocsReorder202507 array, and the destination page (content/docs/observability/sdk/overview.mdx) exists.
  • The change is consistent with the existing redirect pattern in the file and requires no further adjustments.

Confidence Score: 5/5

This is a one-line, additive redirect entry with a verified destination page — safe to merge as-is.

The added entry follows the exact same pattern as the three adjacent python SDK redirects, the destination page exists on disk, and nothing else in the file is touched.

No files require special attention.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A["/docs/sdk/python/sdk-v4 (legacy)"] -->|redirect via mainDocsReorder202507| B["/docs/observability/sdk/overview"]
    C["/docs/sdk/python/sdk-v3 (legacy)"] -->|redirect| B
    D["/docs/sdk/python/example (legacy)"] -->|redirect| B
    E["/docs/sdk/python/low-level-sdk (legacy)"] -->|redirect| B
    B --> F["content/docs/observability/sdk/overview.mdx"]
Loading

Reviews (1): Last reviewed commit: "docs(redirects): add /docs/sdk/python/sd..." | Re-trigger Greptile

langfuse#2917)

Issue langfuse#2917 reports /docs/sdk/python/sdk-v4 returning 404. The
redirects table already has the matching sdk-v3 entry pointing at
the observability overview but is missing sdk-v4. Add the sibling
entry so the legacy URL resolves to the same overview page.
Copy link
Copy Markdown

@claude claude Bot left a comment

Choose a reason for hiding this comment

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

Claude Code Review

This pull request is from a fork — automated review is disabled. A repository maintainer can comment @claude review to run a one-time review.

@vercel
Copy link
Copy Markdown

vercel Bot commented May 19, 2026

@mvanhorn is attempting to deploy a commit to the langfuse Team on Vercel.

A member of the Team first needs to authorize it.

@dosubot dosubot Bot added size:XS This PR changes 0-9 lines, ignoring generated files. docs labels May 19, 2026
@CLAassistant
Copy link
Copy Markdown

CLAassistant commented May 20, 2026

CLA assistant check
All committers have signed the CLA.

@mvanhorn
Copy link
Copy Markdown
Author

@cla-assistant recheck

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Broken link: /docs/sdk/python/sdk-v4

2 participants