Skip to content

build: bump slack sdk and bolt to latest release#22

Merged
mwbrooks merged 5 commits intoai-appsfrom
ale-ai-apps-bump
Oct 7, 2025
Merged

build: bump slack sdk and bolt to latest release#22
mwbrooks merged 5 commits intoai-appsfrom
ale-ai-apps-bump

Conversation

@srtaalej
Copy link
Copy Markdown
Contributor

@srtaalej srtaalej commented Oct 7, 2025 •
edited
Loading

Type of change

  • New feature
  • Bug fix
  • Documentation
  • Build

Summary

update the sdk and bolt version to newest release

Requirements

  • I have ensured the changes I am contributing align with existing patterns and have tested and linted my code
  • I've read and agree to the Code of Conduct

@srtaalej srtaalej changed the title Ale ai apps bump build: bump slack sdk and bolt to latest release Oct 7, 2025
@srtaalej srtaalej self-assigned this Oct 7, 2025
@srtaalej srtaalej requested review from mwbrooks and zimeg and removed request for zimeg October 7, 2025 01:15
@srtaalej srtaalej added bug Something isn't working dependencies Pull requests that update a dependency file labels Oct 7, 2025
zimeg
zimeg reviewed Oct 7, 2025
View reviewed changes
Copy link
Copy Markdown
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.

@srtaalej LGTM but perhaps a quick change related to the manifest before we test this? Hoping to reduce scopes overall needed!

Comment thread listeners/assistant/assistant.py
zimeg
zimeg approved these changes Oct 7, 2025
View reviewed changes
Copy link
Copy Markdown
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.

@srtaalej LGTM! Thanks for removing the scopes unused after these changes. I am optimistic that this makes the sample more intuitive at a glance.

🔍 Perhaps soon we bring search as a replacement?

mwbrooks
mwbrooks approved these changes Oct 7, 2025
View reviewed changes
Copy link
Copy Markdown
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 a ton @srtaalej for updating this app to not require the channel history method! And thank you @zimeg for removing the extra scopes!

🧪 Testing locally works well with all of the features.

@mwbrooks mwbrooks merged commit 2890ab4 into ai-apps Oct 7, 2025
3 checks passed
@mwbrooks mwbrooks deleted the ale-ai-apps-bump branch October 7, 2025 01:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mwbrooks mwbrooks mwbrooks approved these changes

@zimeg zimeg zimeg approved these changes

Assignees

@srtaalej srtaalej

Labels

bug Something isn't working dependencies Pull requests that update a dependency file

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@srtaalej @mwbrooks @zimeg