Skip to content

fix(dashboard): adjust agent Slack overview copy fixes NV-7381#10778

Merged
scopsy merged 1 commit intonextfrom
cursor/adjust-slack-agent-copy-nv7381-5aa4
Apr 19, 2026
Merged

fix(dashboard): adjust agent Slack overview copy fixes NV-7381#10778
scopsy merged 1 commit intonextfrom
cursor/adjust-slack-agent-copy-nv7381-5aa4

Conversation

@scopsy
Copy link
Copy Markdown
Contributor

@scopsy scopsy commented Apr 17, 2026

Summary

The Slack integration Overview for connected agents told users they could DM the bot. Direct messages are not supported in the default flow, which led to confusion when the integration looked active but DMs did not work.

Changes

  • Updated the connected-state copy in slack-agent-integration-guide.tsx to instruct users to mention the bot in a channel it has been invited to, and removed the DM wording.

Testing

  • Not run (copy-only change). Lint check on the edited file is clean.

Linear Issue: NV-7381

Open in Web Open in Cursor 

What changed

Updated the Slack integration "Overview" text displayed when the integration is connected. Removed instructions misleadingly suggesting users could DM the bot directly (a flow not supported in the default configuration) and rephrased the guidance to clarify that users should mention the bot in channels where it has been invited. This resolves confusion around the unsupported DM interaction pattern.

Affected areas

dashboard: Updated the connected-state copy in slack-agent-integration-guide.tsx to provide accurate channel-mention guidance instead of direct message instructions.

Testing

No automated tests added—this is a copy-only change. Lint checks on the modified file passed successfully.

…NV-7381

Co-authored-by: Dima Grossman <dima@grossman.io>
@linear
Copy link
Copy Markdown

linear Bot commented Apr 17, 2026

@netlify
Copy link
Copy Markdown

netlify Bot commented Apr 17, 2026

Deploy preview added

Name Link
🔨 Latest commit 189c519
🔍 Latest deploy log https://app.netlify.com/projects/dashboard-v2-novu-staging/deploys/69e2402f1bb33100080b7438
😎 Deploy Preview https://deploy-preview-10778.dashboard-v2.novu-staging.co
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Apr 17, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 506c3f65-417b-44ae-b2cb-ee48bd8ec007

📥 Commits

Reviewing files that changed from the base of the PR and between 0ad1fe5 and 189c519.

📒 Files selected for processing (1)
  • apps/dashboard/src/components/agents/agent-integration-guides/slack-agent-integration-guide.tsx

📝 Walkthrough

Walkthrough

Updates the connected-state overview text in the Slack agent integration guide by removing direct message instructions and refocusing guidance solely on channel invitations for sending and receiving messages.

Changes

Cohort / File(s) Summary
Slack Integration Guide Text Update
apps/dashboard/src/components/agents/agent-integration-guides/slack-agent-integration-guide.tsx
Modified the "Overview" section text displayed when Slack integration is connected: removed the instruction to "DM it directly" and rephrased to reference only tagging the bot in channels it has been invited to.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@scopsy scopsy marked this pull request as ready for review April 19, 2026 08:21
@scopsy scopsy merged commit 522d12c into next Apr 19, 2026
31 checks passed
@scopsy scopsy deleted the cursor/adjust-slack-agent-copy-nv7381-5aa4 branch April 19, 2026 08:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants