Skip to content

Comments

feat: add markdown_text to chat.{postEphemeral,postMessage,scheduleMessage,update} api methods#1531

Merged
zimeg merged 1 commit intomainfrom
zimeg-feat-client-markdowntext
Nov 11, 2025
Merged

feat: add markdown_text to chat.{postEphemeral,postMessage,scheduleMessage,update} api methods#1531
zimeg merged 1 commit intomainfrom
zimeg-feat-client-markdowntext

Conversation

@zimeg
Copy link
Member

@zimeg zimeg commented Nov 11, 2025

This PR adds the markdown_text attribute to the following methods:

Category

  • bolt (Bolt for Java)
  • slack-api-model (Slack API Data Models)

Notes

These changes are surfaced with bolt too! Though I believe this builder pattern requires the channel be provided:

ctx.say(r -> r.markdownText("**woah**").channel("C0123456789"));

java-slack-sdk/bolt/src/main/java/com/slack/api/bolt/context/builtin/EventContext.java

Lines 94 to 105 in b08ba12

@Override
public ChatPostMessageResponse say(BuilderConfigurator<ChatPostMessageRequest.ChatPostMessageRequestBuilder> request) throws IOException, SlackApiException {
if (isAssistantThreadEvent()) {
ChatPostMessageRequest params = request.configure(ChatPostMessageRequest.builder()).build();
params.setChannel(this.getChannelId());
params.setThreadTs(this.getThreadTs());
params.setMetadata(this.buildMetadata());
return this.client().chatPostMessage(params);
} else {
return super.say(request);
}
}

Requirements

Please read the Contributing guidelines and Code of Conduct before creating this issue or pull request. By submitting, you agree to those rules.

@zimeg zimeg added this to the 1.47.0 milestone Nov 11, 2025
@zimeg zimeg self-assigned this Nov 11, 2025
@zimeg zimeg added enhancement M-T: A feature request for new functionality project:bolt project:slack-api-client project:slack-api-client labels Nov 11, 2025
@codecov
Copy link

codecov bot commented Nov 11, 2025 •
edited
Loading

Codecov Report

❌ Patch coverage is 88.88889% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 72.89%. Comparing base (b08ba12) to head (54b9ff8).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
...java/com/slack/api/methods/RequestFormBuilder.java 88.88% 0 Missing and 1 partial ⚠️
Additional details and impacted files 
@@             Coverage Diff              @@
##               main    #1531      +/-   ##
============================================
+ Coverage     72.86%   72.89%   +0.02%     
+ Complexity     4414     4413       -1     
============================================
  Files           477      477              
  Lines         14073    14081       +8     
  Branches       1473     1473              
============================================
+ Hits          10254    10264      +10     
+ Misses         2946     2944       -2     
  Partials        873      873

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

WilliamBergamin
WilliamBergamin approved these changes Nov 11, 2025
View reviewed changes
Copy link
Contributor

@WilliamBergamin WilliamBergamin left a comment

Choose a reason for hiding this comment

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

Thanks for working on this 💯

@zimeg
Copy link
Member Author

zimeg commented Nov 11, 2025

@WilliamBergamin And thank you for a fast review! ☕ ✨

@zimeg zimeg merged commit 610d528 into main Nov 11, 2025
6 checks passed
@zimeg zimeg deleted the zimeg-feat-client-markdowntext branch November 11, 2025 16:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@WilliamBergamin WilliamBergamin WilliamBergamin approved these changes

@mwbrooks mwbrooks Awaiting requested review from mwbrooks

@srtaalej srtaalej Awaiting requested review from srtaalej

Assignees

@zimeg zimeg

Labels

enhancement M-T: A feature request for new functionality project:bolt project:slack-api-client project:slack-api-client

Projects

None yet

Milestone

1.47.0

Development

Successfully merging this pull request may close these issues.

2 participants

@zimeg @WilliamBergamin