Skip to content

fix: SCIM links & remove incorrect docs - #753

Merged
Brian Vander Schaaf (bvs-langchain) merged 4 commits into
mainfrom
brian/ent-100-okta-follow-up-updates
Oct 6, 2025
Merged

fix: SCIM links & remove incorrect docs#753
Brian Vander Schaaf (bvs-langchain) merged 4 commits into
mainfrom
brian/ent-100-okta-follow-up-updates

Conversation

@bvs-langchain

@bvs-langchain Brian Vander Schaaf (bvs-langchain) commented Oct 6, 2025

Copy link
Copy Markdown
Contributor

Overview

  • fix broken links
  • remove incorrect SCIM setup steps (these were for without using OIN, but Okta requires using OIN)
  • add missing supported features for Okta SCIM

Type of change

Type: Update existing documentation

Related issues/PRs

  • GitHub issue:
  • Feature PR:
  • Linear issue: part of ent-100
  • 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 6, 2025 18:38
@github-actions github-actions Bot added the langsmith For docs changes to LangSmith label Oct 6, 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 SCIM-related documentation issues by correcting broken internal links and removing incorrect configuration details. The changes improve the accuracy and usability of the SCIM setup instructions.

  • Fixed broken internal link reference to SCIM setup section
  • Removed incorrect API header parameter from SCIM token generation
  • Updated Okta SCIM configuration steps to reflect current UI and removed outdated Base URL configuration
  • Minor formatting improvements with blank line additions/removals

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
src/langsmith/user-management.mdx Fixed SCIM link reference, removed incorrect API header, updated Okta configuration steps, and improved formatting
src/langsmith/self-host-sso.mdx Removed extra blank lines for cleaner formatting

Comment thread src/langsmith/user-management.mdx Outdated
@github-actions

github-actions Bot commented Oct 6, 2025

Copy link
Copy Markdown
Contributor

Preview ID generated: preview-briane-1759776159-85942db

@github-actions

github-actions Bot commented Oct 6, 2025

Copy link
Copy Markdown
Contributor

Preview ID generated: preview-briane-1759776372-35cb28f

Copilot AI review requested due to automatic review settings October 6, 2025 18:47

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.

@github-actions

github-actions Bot commented Oct 6, 2025

Copy link
Copy Markdown
Contributor

Preview ID generated: preview-briane-1759776464-b66e377

@bvs-langchain
Brian Vander Schaaf (bvs-langchain) merged commit 78bdce3 into main Oct 6, 2025
11 checks passed
@bvs-langchain
Brian Vander Schaaf (bvs-langchain) deleted the brian/ent-100-okta-follow-up-updates branch October 6, 2025 18:50
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.

3 participants