Skip to content

docs(slack): add missing DM scopes to manifest#29999

Merged
Takhoffman merged 2 commits intoopenclaw:mainfrom
JcMinarro:fix-slack-manifest
Mar 1, 2026
Merged

docs(slack): add missing DM scopes to manifest#29999
Takhoffman merged 2 commits intoopenclaw:mainfrom
JcMinarro:fix-slack-manifest

Conversation

@JcMinarro
Copy link
Contributor

$(cat <<EOF

Summary

Add missing Slack OAuth scopes to the app manifest documentation:

  • im:read - Read direct messages
  • im:write - Write direct messages
  • mpim:read - Read multi-party direct messages
  • mpim:write - Write multi-party direct messages

These scopes are required for the bot to send messages in DMs and group DMs. Without them, OpenClaw gets an error when trying to respond to direct messages.

Related

The zh-CN translation already had these scopes; this updates the English version to match.
EOF
)

Copilot AI review requested due to automatic review settings February 28, 2026 18:39
@openclaw-barnacle openclaw-barnacle bot added docs Improvements or additions to documentation channel: slack Channel integration: slack size: XS labels Feb 28, 2026
Copy link
Contributor

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

Updates the Slack channel documentation to include missing OAuth scopes needed for direct message (IM) and multi-party direct message (MPIM) functionality, aligning the English manifest example with the existing zh-CN translation and with runtime behavior that uses DM-related Slack APIs.

Changes:

  • Add im:read and im:write to the documented Slack bot scopes.
  • Add mpim:read and mpim:write to the documented Slack bot scopes.

@greptile-apps
Copy link
Contributor

greptile-apps bot commented Feb 28, 2026

Greptile Summary

Adds four missing Slack OAuth scopes (im:read, im:write, mpim:read, mpim:write) to the English app manifest documentation. These scopes enable direct message and group DM functionality, aligning the English docs with the existing Chinese translation that already included them.

  • Scopes correctly placed in alphabetical order within the bot scopes array
  • JSON formatting is correct with proper comma placement
  • These are legitimate Slack OAuth scopes required for DM/group DM read and write operations

Confidence Score: 5/5

  • This PR is safe to merge with no risk
  • Documentation-only change adding legitimate Slack OAuth scopes to the manifest example. No code changes, no logic changes, and aligns with existing Chinese translation
  • No files require special attention

Last reviewed commit: 95e94bd

@Takhoffman Takhoffman force-pushed the fix-slack-manifest branch from 95e94bd to a45cfab Compare March 1, 2026 17:25
@Takhoffman Takhoffman merged commit b9e07ad into openclaw:main Mar 1, 2026
9 checks passed
@Takhoffman
Copy link
Contributor

PR #29999 - docs(slack): add missing DM scopes to manifest (#29999)

Merged via squash.

  • Merge commit: b9e07ad
  • Verified: pnpm install --frozen-lockfile, pnpm build, pnpm check, pnpm test:macmini
  • Changes made:
    M CHANGELOG.md
    M docs/channels/slack.md
  • Why these changes were made:
    Added required changelog coverage so this docs fix can land under the autoland changelog guardrail.
  • Changelog: CHANGELOG.md updated=true required=true opt_out=false

Thanks @JcMinarro!

zooqueen added a commit to hanzoai/bot that referenced this pull request Mar 1, 2026
ansh pushed a commit to vibecode/openclaw that referenced this pull request Mar 2, 2026
…@JcMinarro

Verified:
- pnpm install --frozen-lockfile
- pnpm build
- pnpm check
- pnpm test:macmini

Co-authored-by: JcMinarro <4047514+JcMinarro@users.noreply.github.com>
Co-authored-by: Tak Hoffman <781889+Takhoffman@users.noreply.github.com>
steipete pushed a commit to Sid-Qin/openclaw that referenced this pull request Mar 2, 2026
…@JcMinarro

Verified:
- pnpm install --frozen-lockfile
- pnpm build
- pnpm check
- pnpm test:macmini

Co-authored-by: JcMinarro <4047514+JcMinarro@users.noreply.github.com>
Co-authored-by: Tak Hoffman <781889+Takhoffman@users.noreply.github.com>
safzanpirani pushed a commit to safzanpirani/clawdbot that referenced this pull request Mar 2, 2026
…@JcMinarro

Verified:
- pnpm install --frozen-lockfile
- pnpm build
- pnpm check
- pnpm test:macmini

Co-authored-by: JcMinarro <4047514+JcMinarro@users.noreply.github.com>
Co-authored-by: Tak Hoffman <781889+Takhoffman@users.noreply.github.com>
steipete pushed a commit to Sid-Qin/openclaw that referenced this pull request Mar 2, 2026
…@JcMinarro

Verified:
- pnpm install --frozen-lockfile
- pnpm build
- pnpm check
- pnpm test:macmini

Co-authored-by: JcMinarro <4047514+JcMinarro@users.noreply.github.com>
Co-authored-by: Tak Hoffman <781889+Takhoffman@users.noreply.github.com>
amitmiran137 pushed a commit to amitmiran137/openclaw that referenced this pull request Mar 2, 2026
…@JcMinarro

Verified:
- pnpm install --frozen-lockfile
- pnpm build
- pnpm check
- pnpm test:macmini

Co-authored-by: JcMinarro <4047514+JcMinarro@users.noreply.github.com>
Co-authored-by: Tak Hoffman <781889+Takhoffman@users.noreply.github.com>
robertchang-ga pushed a commit to robertchang-ga/openclaw that referenced this pull request Mar 2, 2026
…@JcMinarro

Verified:
- pnpm install --frozen-lockfile
- pnpm build
- pnpm check
- pnpm test:macmini

Co-authored-by: JcMinarro <4047514+JcMinarro@users.noreply.github.com>
Co-authored-by: Tak Hoffman <781889+Takhoffman@users.noreply.github.com>
hanqizheng pushed a commit to hanqizheng/openclaw that referenced this pull request Mar 2, 2026
…@JcMinarro

Verified:
- pnpm install --frozen-lockfile
- pnpm build
- pnpm check
- pnpm test:macmini

Co-authored-by: JcMinarro <4047514+JcMinarro@users.noreply.github.com>
Co-authored-by: Tak Hoffman <781889+Takhoffman@users.noreply.github.com>
execute008 pushed a commit to execute008/openclaw that referenced this pull request Mar 2, 2026
…@JcMinarro

Verified:
- pnpm install --frozen-lockfile
- pnpm build
- pnpm check
- pnpm test:macmini

Co-authored-by: JcMinarro <4047514+JcMinarro@users.noreply.github.com>
Co-authored-by: Tak Hoffman <781889+Takhoffman@users.noreply.github.com>
hughdidit pushed a commit to hughdidit/DAISy-Agency that referenced this pull request Mar 3, 2026
…@JcMinarro

Verified:
- pnpm install --frozen-lockfile
- pnpm build
- pnpm check
- pnpm test:macmini

Co-authored-by: JcMinarro <4047514+JcMinarro@users.noreply.github.com>
Co-authored-by: Tak Hoffman <781889+Takhoffman@users.noreply.github.com>
(cherry picked from commit b9e07ad)

# Conflicts:
#	CHANGELOG.md
dorgonman pushed a commit to kanohorizonia/openclaw that referenced this pull request Mar 3, 2026
…@JcMinarro

Verified:
- pnpm install --frozen-lockfile
- pnpm build
- pnpm check
- pnpm test:macmini

Co-authored-by: JcMinarro <4047514+JcMinarro@users.noreply.github.com>
Co-authored-by: Tak Hoffman <781889+Takhoffman@users.noreply.github.com>
sachinkundu pushed a commit to sachinkundu/openclaw that referenced this pull request Mar 6, 2026
…@JcMinarro

Verified:
- pnpm install --frozen-lockfile
- pnpm build
- pnpm check
- pnpm test:macmini

Co-authored-by: JcMinarro <4047514+JcMinarro@users.noreply.github.com>
Co-authored-by: Tak Hoffman <781889+Takhoffman@users.noreply.github.com>
zooqueen pushed a commit to hanzoai/bot that referenced this pull request Mar 6, 2026
…@JcMinarro

Verified:
- pnpm install --frozen-lockfile
- pnpm build
- pnpm check
- pnpm test:macmini

Co-authored-by: JcMinarro <4047514+JcMinarro@users.noreply.github.com>
Co-authored-by: Tak Hoffman <781889+Takhoffman@users.noreply.github.com>
zooqueen added a commit to hanzoai/bot that referenced this pull request Mar 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

channel: slack Channel integration: slack docs Improvements or additions to documentation size: XS

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants