Skip to content

fix: Okta docs instructions - #786

Merged
Brian Vander Schaaf (bvs-langchain) merged 1 commit into
mainfrom
brian/ent-100-okta-follow-up-typo
Oct 8, 2025
Merged

fix: Okta docs instructions#786
Brian Vander Schaaf (bvs-langchain) merged 1 commit into
mainfrom
brian/ent-100-okta-follow-up-typo

Conversation

@bvs-langchain

Copy link
Copy Markdown
Contributor

Overview

  • reference LangSmithURL configuration for Okta, as this is required for SCIM
  • fix location of Okta supported features

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 8, 2025 12:49
@github-actions github-actions Bot added the langsmith For docs changes to LangSmith label Oct 8, 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 Okta documentation instructions to improve accuracy and clarity of the SCIM configuration process. It addresses two key issues: missing required configuration information and misplaced feature listings.

  • Adds reference to required LangSmithURL configuration for Okta SCIM setup
  • Moves "Import Users" and "Import Groups" features from general Okta section to the specific SCIM features list where they belong

@github-actions

github-actions Bot commented Oct 8, 2025

Copy link
Copy Markdown
Contributor

Preview ID generated: preview-briane-1759927783-5ed4158


**Step 2: Configure API Integration**

1. In the General tab, ensure the `LangSmithUrl` is filled in according to the instructions from [Step 1](#add-application-okta-oin)

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.

Suggested change
1. In the General tab, ensure the `LangSmithUrl` is filled in according to the instructions from [Step 1](#add-application-okta-oin)
1. In the General tab, ensure the `LangSmithUrl` is filled in according to the instructions from [Step 1](#add-application-okta-oin).

@bvs-langchain
Brian Vander Schaaf (bvs-langchain) merged commit 520c9d8 into main Oct 8, 2025
11 checks passed
@bvs-langchain
Brian Vander Schaaf (bvs-langchain) deleted the brian/ent-100-okta-follow-up-typo branch October 8, 2025 12:58
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