Skip to content

fix(slack): let Block Kit messages carry notification text - #10018

Merged
waleedlatif1 merged 1 commit into
stagingfrom
fix/slack-blocks-fallback-text
Nov 13, 2026
Merged

waleedlatif1 merged 1 commit into
stagingfrom
fix/slack-blocks-fallback-text

Conversation

@waleedlatif1

Copy link
Copy Markdown
Collaborator

Summary

  • In Block Kit mode the Slack block hid its message text field and always sent ' ' as text, so every Block Kit message pushed a blank notification preview
  • Send, ephemeral, scheduled and update now keep their text field visible in Block Kit mode as optional notification and fallback text; it is still required only in plain-text mode
  • The params mapping already forwarded text/updateText when set, so a filled field now reaches Slack

Type of Change

  • Bug fix

Testing

blocks/blocks/slack.test.ts, blocks/blocks/slack-resources.test.ts, blocks/blocks.test.ts, check-canvas-sentences for slack and slack_v2, biome on the file

Checklist

  • Code follows project style guidelines
  • Self-reviewed my changes
  • Tests added/updated and passing (new tests pass the test-audit authoring gate)
  • No new warnings introduced
  • I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

@vercel

vercel Bot commented Nov 13, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Actions Updated
docs Skipped Skipped Nov 13, 2026 10:31pm UTC

Request Review

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

No issues found across 1 file

Confidence score: 5/5

  • Automated review surfaced no issues in the provided summaries.
  • No files require special attention.

Turn on auto-fix | Re-trigger cubic

@greptile-apps

greptile-apps Bot commented Nov 13, 2026 •

Copy link
Copy Markdown
Contributor

RetriggerView in GreptileConfidence Score: 5/5

[Medium impact] The PR appears safe to merge.

Summary

Keeps text and updateText visible in Block Kit mode for sending, ephemeral messages, scheduled messages, and updates.

  • Block Kit messages can include notification text.

Reviews (2) · Last reviewed commit: "fix(slack): let Block Kit messages carry..." · Reviewed by Greptile

Comment thread apps/sim/blocks/blocks/slack.ts Resolved
@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@greptile

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@cubic-dev-ai review this PR

@cubic-dev-ai

cubic-dev-ai Bot commented Nov 13, 2026

Copy link
Copy Markdown
Contributor

@cubic-dev-ai review this PR

@waleedlatif1 I have started the AI code review. It will take a few minutes to complete.

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

No issues found across 1 file

Confidence score: 5/5

  • Automated review surfaced no issues in the provided summaries.
  • No files require special attention.

Turn on auto-fix | Re-trigger cubic

@waleedlatif1
waleedlatif1 merged commit b5b8ec3 into staging Nov 13, 2026
80 checks passed
@waleedlatif1
waleedlatif1 deleted the fix/slack-blocks-fallback-text branch November 14, 2026 00:29

This branch was previously deployed

1 inactive deployment
Preview — 35f4ae6a Deployed Nov 13, 2026 by vercel[bot]
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.

1 participant