Skip to content

fix broken links in quickstart - #782

Merged
Mason Daugherty (mdrxy) merged 4 commits into
mainfrom
fix_quickstart_links
Oct 7, 2025
Merged

fix broken links in quickstart#782
Mason Daugherty (mdrxy) merged 4 commits into
mainfrom
fix_quickstart_links

Conversation

@eyurtsev

Copy link
Copy Markdown
Collaborator

Overview

Type of change

Type: [Replace with: New documentation page / Update existing documentation / Fix typo/bug/link/formatting / Remove outdated content / Other]

Related issues/PRs

  • GitHub issue:
  • Feature PR:
  • Linear issue:
  • Slack thread:

Checklist

  • I have read the contributing guidelines
  • I have tested my changes locally using docs dev
  • All code examples have been tested and work correctly
  • I have used root relative paths for internal links
  • I have updated navigation in src/docs.json if needed
  • I have gotten approval from the relevant reviewers
  • (Internal team members only / optional) I have created a preview deployment using the Create Preview Branch workflow

Additional notes

Copilot AI review requested due to automatic review settings October 7, 2025 21:26
@github-actions github-actions Bot added langchain For docs changes to LangChain oss labels Oct 7, 2025

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 fixes broken internal links in the LangChain quickstart documentation by updating three link references to point to the correct documentation paths.

  • Updates memory-related link from /oss/langchain/memory to /oss/langchain/short-term-memory
  • Converts @tool decorator reference from plain text to a proper external link
  • Updates production memory guide link to point to the correct section

Comment thread src/oss/langchain/quickstart.mdx Outdated
<Info>
In production, use a persistent checkpointer that saves to a database.
See [add and manage memory](/oss/python/langgraph/add-memory) for more details.
See [add and manage memory](/oss/python/langchain/short-term-memory#in-production) for more details.

Copilot AI Oct 7, 2025

Copy link

Choose a reason for hiding this comment

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

The link path appears to mix LangChain and LangGraph paths. Based on the context of memory management in LangChain, this should likely be /oss/langchain/short-term-memory#in-production (without the /python segment) to match the pattern of other LangChain documentation links in this file.

Suggested change
See [add and manage memory](/oss/python/langchain/short-term-memory#in-production) for more details.
See [add and manage memory](/oss/langchain/short-term-memory#in-production) for more details.

Copilot uses AI. Check for mistakes.
Comment thread src/oss/langchain/quickstart.mdx Outdated
@github-actions

github-actions Bot commented Oct 7, 2025

Copy link
Copy Markdown
Contributor

Preview ID generated: preview-fixqui-1759872422-53ce7aa

@github-actions github-actions Bot added python For content related to the Python version of LangChain projects docs-infra labels Oct 7, 2025
@github-actions

github-actions Bot commented Oct 7, 2025

Copy link
Copy Markdown
Contributor

Preview ID generated: preview-fixqui-1759872572-bf8d642

Copilot AI review requested due to automatic review settings October 7, 2025 22:31

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull Request Overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated no new comments.

@mdrxy
Mason Daugherty (mdrxy) merged commit 3161d65 into main Oct 7, 2025
9 checks passed
@mdrxy
Mason Daugherty (mdrxy) deleted the fix_quickstart_links branch October 7, 2025 22:31
@github-actions

github-actions Bot commented Oct 7, 2025

Copy link
Copy Markdown
Contributor

Preview ID generated: preview-fixqui-1759876313-a4ea590

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

Labels

docs-infra langchain For docs changes to LangChain oss python For content related to the Python version of LangChain projects

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants