Skip to content

Conversation

romain-priour-lc
Copy link
Contributor

@romain-priour-lc romain-priour-lc commented Sep 30, 2025

Overview

Customers didn't know to include X-tenant-Id header. Doesn't hurt to mention it in the section where we explain what each API key is.

Type of change

Type:
Update existing documentation

To automatically close an issue when this PR is merged, use closing keywords:

  • Closes INF-868

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

@github-actions github-actions bot added the langsmith For docs changes to LangSmith label Sep 30, 2025
Copy link

Preview ID generated: preview-romain-1759204940-38c240f

Copy link

Preview ID generated: preview-romain-1759205276-b5d40a5

Copy link
Contributor

@katmayb katmayb left a comment

Choose a reason for hiding this comment

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

Non-blocking suggestion, but lgtm!

Co-authored-by: Kathryn May <44557882+katmayb@users.noreply.github.com>
Copy link

github-actions bot commented Oct 2, 2025

Preview ID generated: preview-romain-1759367400-8e495c7

@romain-priour-lc romain-priour-lc merged commit 28f5d56 into main Oct 2, 2025
11 checks passed
@romain-priour-lc romain-priour-lc deleted the romain/add-tenant-id-note-service-key branch October 2, 2025 01:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
langsmith For docs changes to LangSmith
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants