fix: correct Slack webhook example code in documentation #21295
Merged
+5
−5
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
All contributors have signed the CLA ✍️ ✅ |
578f3e9 to
bd1f0d3
Compare
Contributor
Author
|
I have read the CLA Document and I hereby sign the CLA |
cdrci2
added a commit
to coder/cla
that referenced
this pull request
Dec 16, 2025
- Fix ReferenceError by changing 'text: body' to 'text: body_markdown' on line 118 (the variable 'body' was never defined; should use 'body_markdown' from req.body) - Fix invalid_blocks API error by changing header block text type from 'mrkdwn' to 'plain_text' on line 122 (Slack's header block type only supports plain_text, not mrkdwn) - Rename 'title_markdown' to 'title' since the header block uses plain_text type, not markdown (variable names should reflect their actual content) These errors were preventing webhook messages from being sent to Slack.
bd1f0d3 to
28b5ca6
Compare
Contributor
There was a problem hiding this comment.
Pull request overview
This PR fixes two bugs in the Slack webhook integration example code that would prevent it from working correctly. The first fix resolves a ReferenceError by using the correct variable name (body_markdown instead of undefined body), and the second fix corrects the Slack API usage by changing the header block text type to plain_text (the only supported type for headers) instead of mrkdwn.
- Fixed undefined variable reference in Slack message text field
- Corrected Slack header block to use
plain_texttype as required by Slack's Block Kit API - Updated variable naming from
title_markdowntotitlefor consistency with plain text usage
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
mtojek
approved these changes
Dec 17, 2025
Member
mtojek
left a comment
There was a problem hiding this comment.
Thank you for your contribution!
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
Fixes two bugs in the Slack webhook example code that prevented messages from being sent to Slack:
ReferenceError: body is not defined - Changed
text: bodytotext: body_markdown(line 118)bodywas never defined in the webhook handlerbody_markdownwhich is extracted from the request body on line 92Invalid header block type - Changed header block text type from
mrkdwntoplain_text(line 122)Related Issue
Fixes #21294