Skip to content

chore: add PrivateLink docs#3610

Merged
joaquin-borggio-lc merged 4 commits into
mainfrom
joaquin/add-privatelink-docs
Apr 17, 2026
Merged

chore: add PrivateLink docs#3610
joaquin-borggio-lc merged 4 commits into
mainfrom
joaquin/add-privatelink-docs

Conversation

@joaquin-borggio-lc

@joaquin-borggio-lc joaquin-borggio-lc commented Apr 16, 2026

Copy link
Copy Markdown
Contributor

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, including the language policy
  • 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

(Internal team members only / optional): Create a preview deployment as necessary using the Create Preview Branch workflow

Additional notes

@github-actions github-actions Bot added langsmith For docs changes to LangSmith internal labels Apr 16, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Thanks for opening a docs PR, joaquin-borggio-lc! When it's ready for review, please add the relevant reviewers:

  • @katmayb or @fjmorris (LangSmith)

@github-actions

Copy link
Copy Markdown
Contributor

Mintlify preview ID generated: preview-joaqui-1776378023-411a3a8

@github-actions

Copy link
Copy Markdown
Contributor

Mintlify preview ID generated: preview-joaqui-1776438011-a4218af

@github-actions

Copy link
Copy Markdown
Contributor

Mintlify preview ID generated: preview-joaqui-1776441687-6716bcf

Comment thread src/langsmith/cloud.mdx Outdated
```
</CodeGroup>

**3. Wait for acceptance.** LangChain will accept the connection. The endpoint status will change from `pendingAcceptance` to `available`. Allow a few minutes after acceptance for the data plane to fully propagate before testing connectivity.

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.

what does "for the data plane to fully propagate" mean here?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Good catch the wording is off. It should be for the change to propagate

@github-actions

Copy link
Copy Markdown
Contributor

Mintlify preview ID generated: preview-joaqui-1776442381-7cdb21c

@katmayb Kathryn May (katmayb) 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.

Looks good, you can merge as is, but H5 headers don't provide a link — I'll sort this out once you've merged though 👍

@joaquin-borggio-lc

Copy link
Copy Markdown
Contributor Author

Looks good, you can merge as is, but H5 headers don't provide a link — I'll sort this out once you've merged though 👍

I dont think we need link for H5 headers here, but lmk if you think we need later

@joaquin-borggio-lc
joaquin-borggio-lc merged commit 84281a9 into main Apr 17, 2026
19 checks passed
@joaquin-borggio-lc
joaquin-borggio-lc deleted the joaquin/add-privatelink-docs branch April 17, 2026 17:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

internal langsmith For docs changes to LangSmith

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants