Skip to content

Conversation

bvs-langchain
Copy link
Contributor

@bvs-langchain bvs-langchain commented Oct 1, 2025

Overview

  • include option of updating sharing settings in the UI since we do support this
Screenshot 2025-10-01 at 1 51 45 PM

Type of change

Type: Update existing documentation

Related issues/PRs

  • GitHub issue:
  • Feature PR:
  • Linear issue: closes ent-133
  • 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 Copilot AI review requested due to automatic review settings October 1, 2025 17:52
Copy link
Contributor

@Copilot Copilot AI left a comment

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 updates the security settings documentation to include information about the UI option for managing shared resource settings. The change adds clarification that these settings can be updated both through the UI and via API, along with important warnings about the organization-wide impact.

  • Added a warning callout highlighting that sharing settings affect all organization resources
  • Included reference to the UI location for updating settings (Settings > Shared tab)
  • Maintained existing API documentation while adding the UI alternative

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

github-actions bot commented Oct 1, 2025

Preview ID generated: preview-briane-1759341208-a912c16

Co-authored-by: Kathryn May <44557882+katmayb@users.noreply.github.com>
@Copilot Copilot AI review requested due to automatic review settings October 1, 2025 18:17
@bvs-langchain bvs-langchain enabled auto-merge (squash) October 1, 2025 18:17
Copy link
Contributor

@Copilot Copilot AI left a comment

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 1 out of 1 changed files in this pull request and generated 1 comment.

Updating these settings affects **all resources in the organization**.
</Warning>

You can update these settings under the **Settings > Shared** tab for a workspace, or via API:
Copy link

Copilot AI Oct 1, 2025

Choose a reason for hiding this comment

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

The navigation path should be more specific. Consider clarifying whether 'workspace' refers to the LangSmith workspace and provide the complete navigation path from the main interface.

Suggested change
You can update these settings under the **Settings > Shared** tab for a workspace, or via API:
You can update these settings in the LangSmith UI by navigating to **Main menu > [Your Workspace] > Settings > Shared** tab, or via API:

Copilot uses AI. Check for mistakes.

@bvs-langchain bvs-langchain merged commit 3c2f604 into main Oct 1, 2025
10 of 11 checks passed
@bvs-langchain bvs-langchain deleted the brian/ent-133-security-settings branch October 1, 2025 18:19
Copy link

github-actions bot commented Oct 1, 2025

Preview ID generated: preview-briane-1759343031-5e8d6aa

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