Skip to content

Conversation

@lukegalbraithrussell
Copy link
Contributor

Summary

Adds a streaming messages section to sending message page

Testing

Category

  • slack_bolt.App and/or its core components
  • slack_bolt.async_app.AsyncApp and/or its core components
  • Adapters in slack_bolt.adapter
  • Document pages under /docs
  • Others

Requirements

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

  • I've read and understood the Contributing Guidelines and have done my best effort to follow them.
  • I've read and agree to the Code of Conduct.
  • I've run ./scripts/install_all_and_run_tests.sh after making the changes.

@codecov
Copy link

codecov bot commented Oct 3, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 91.03%. Comparing base (7cedaac) to head (221397a).
⚠️ Report is 20 commits behind head on ai-apps.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@             Coverage Diff             @@
##           ai-apps    #1384      +/-   ##
===========================================
+ Coverage    91.00%   91.03%   +0.02%     
===========================================
  Files          222      222              
  Lines         7512     7514       +2     
===========================================
+ Hits          6836     6840       +4     
+ Misses         676      674       -2     

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

Copy link
Contributor

Choose a reason for hiding this comment

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

LGTM!

@zimeg zimeg added docs Improvements or additions to documentation semver:minor labels Oct 3, 2025
Copy link
Member

@zimeg zimeg left a comment

Choose a reason for hiding this comment

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

@lukegalbraithrussell LGTM! A few comments on links and recommending next steps popped out to me in a reading, but the reading was good for me. Your call on if these suggestions are decent! 📚 ✨

Copy link
Member

@mwbrooks mwbrooks 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 adding the chat_*Stream method concept docs!

@mwbrooks mwbrooks changed the title docs: adds ai apps docs docs: adds streaming messages section to sending message page Oct 3, 2025
@lukegalbraithrussell lukegalbraithrussell merged commit 36ddda7 into ai-apps Oct 4, 2025
14 checks passed
@lukegalbraithrussell lukegalbraithrussell deleted the docs-ai-apps branch October 4, 2025 01:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs Improvements or additions to documentation semver:minor

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants