-
Notifications
You must be signed in to change notification settings - Fork 831
feat: add slack app install page for app verification #1260
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
Mintlify preview ID generated: preview-samadd-1762200334-16bb8c1 |
There was a problem hiding this 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 adds a new documentation page for installing the LangSmith Agent Builder Slack app, enabling users to connect their agents to Slack workspaces. The PR includes:
- A new installation guide for the Slack integration
- Updates to the navigation structure to include the new page
Reviewed Changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| src/langsmith/agent-builder-slack-install.mdx | New documentation page explaining how to install and configure the LangSmith Agent Builder Slack app |
| src/docs.json | Adds the new Slack installation page to the Agent Builder tab navigation |
|
Mintlify preview ID generated: preview-samadd-1762202695-1e80164 |
There was a problem hiding this 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 2 out of 2 changed files in this pull request and generated 2 comments.
katmayb
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Left some suggestions, but otherwise lgtm.
Co-authored-by: Kathryn May <44557882+katmayb@users.noreply.github.com>
Co-authored-by: Kathryn May <44557882+katmayb@users.noreply.github.com>
Co-authored-by: Kathryn May <44557882+katmayb@users.noreply.github.com>
65c7d57 to
093334a
Compare
There was a problem hiding this 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 2 out of 2 changed files in this pull request and generated 4 comments.
| After installation, your agents will be able to: | ||
|
|
||
| - Send direct messages. | ||
| - Post to channels. | ||
| - Read thread messages. |
Copilot
AI
Nov 3, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The heading 'How to install' and the introductory sentence 'To install the LangSmith Agent Builder for Slack:' are redundant. Following documentation best practices from the coding guidelines, consider removing the introductory sentence and letting the heading speak for itself, similar to how the PagerDuty integration documentation (alerts-pagerduty.mdx) handles procedural content.
|
Mintlify preview ID generated: preview-samadd-1762205100-7ef8523 |
|
Mintlify preview ID generated: preview-samadd-1762205220-0d0928f |
|
Mintlify preview ID generated: preview-samadd-1762205295-41374ab |
|
Mintlify preview ID generated: preview-samadd-1762205366-fa5f7fa |
as part of the review process with the Slack App marketplace, we are supposed to have a page somewhere that has installation instructions for our Slack app. I figured Agent Builder docs was the easiest place to put that.