Skip to content
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

custom slack app #93

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

custom slack app #93

wants to merge 1 commit into from

Conversation

benauriemma
Copy link
Contributor

@benauriemma benauriemma commented Oct 25, 2024

First draft. Need to review myself, make sure it looks good, and add the picture


Important

Adds guide for creating Custom Slack Apps that connect to specific Copilots, with manual configuration required during beta.

  • Feature:
    • Adds guide for creating Custom Slack Apps in copilots.mdx.
    • Custom Slack Apps connect to specific Copilots with custom name and profile picture.
  • Configuration:
    • Step 1: Create Slack app using provided JSON manifest template.
    • Step 2: Share app information with Credal for manual configuration.
  • Beta Limitations:
    • Configuration is manual and requires contacting Credal support.

This description was created by Ellipsis for acbe3c9. It will automatically update as commits are pushed.

Copy link

@ellipsis-dev ellipsis-dev bot left a comment

Choose a reason for hiding this comment

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

👍 Looks good to me! Reviewed everything up to acbe3c9 in 10 seconds

More details
  • Looked at 113 lines of code in 1 files
  • Skipped 1 files when reviewing.
  • Skipped posting 3 drafted comments based on config settings.
1. fern/docs/pages/guides/copilots.mdx:30
  • Draft comment:
    Consider providing an example or explanation for placeholders [$NAME], [$DESCRIPTION], and [$LONG_DESCRIPTION] to guide users on what values to use.
  • Reason this comment was not posted:
    Confidence changes required: 50%
    The manifest template uses placeholders that need to be replaced by the user. It would be helpful to provide a clear example or explanation of what these placeholders represent.
2. fern/docs/pages/guides/copilots.mdx:63
  • Draft comment:
    Ensure that all listed scopes are necessary for the app's functionality to minimize security risks. This applies to all scopes listed here.
  • Reason this comment was not posted:
    Confidence changes required: 50%
    The manifest JSON structure is correct, but it's important to ensure that the scopes and events are necessary and minimal for security reasons.
3. fern/docs/pages/guides/copilots.mdx:89
  • Draft comment:
    Verify that all listed bot events are necessary for the app's functionality to minimize security risks. This applies to all events listed here.
  • Reason this comment was not posted:
    Confidence changes required: 50%
    The event subscriptions section should be verified to ensure that only necessary events are subscribed to, reducing potential security risks.

Workflow ID: wflow_wgbA47fqTdizCv7k


You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.

Copy link
Member

@jackfischer jackfischer left a comment

Choose a reason for hiding this comment

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

Looks really good. I'm splitting out the copilots section into multiple pages - this will probably go in another page, but is fine for now

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants