Skip to content

Docs: Update Bolt Python Assistant doc to match JS structure. - #1239

Merged
technically-tracy merged 3 commits into
mainfrom
tracy-assistant-structure-py
Jan 30, 2025
Merged

Docs: Update Bolt Python Assistant doc to match JS structure.#1239
technically-tracy merged 3 commits into
mainfrom
tracy-assistant-structure-py

Conversation

@technically-tracy

@technically-tracytechnically-tracy commented Jan 28, 2025•
edited
Loading

Copy link
Copy Markdown
Contributor

Summary

Update Bolt Python Assistant doc to match JS structure.

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

lukegalbraithrussell
lukegalbraithrussell reviewed Jan 28, 2025
View reviewed changes
Comment threaddocs/content/basic/assistant.md Outdated
seratch
seratch approved these changes Jan 28, 2025
View reviewed changes

@seratchseratch 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.

Thank you; this is great!

@seratchseratch added the docs Improvements or additions to documentation label Jan 28, 2025
@seratchseratch added this to the 1.22.1 milestone Jan 28, 2025
lukegalbraithrussell
lukegalbraithrussell reviewed Jan 28, 2025
View reviewed changes
Comment threaddocs/content/basic/assistant.md Outdated
@WilliamBergamin

WilliamBergamin commented Jan 28, 2025•
edited
Loading

Copy link
Copy Markdown
Contributor

Not sure whats going on with these tests 🤔 let me know if you would like some help with them, I don't think you changes have anything to do with this

@technically-tracy

technically-tracy commented Jan 29, 2025

Copy link
Copy Markdown
ContributorAuthor

Not sure whats going on with these tests 🤔 let me know if you would like some help with them, I don't think you changes have anything to do with this

Yeah, I have no idea 😬 I tried to re-run them, but they're still failing.

@WilliamBergaminWilliamBergamin mentioned this pull request Jan 29, 2025
Merged
8 tasks
@codecov

codecovBot commented Jan 29, 2025•
edited
Loading

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 90.96%. Comparing base (276eae6) to head (c8cf99f).
Report is 1 commits behind head on main.

Additional details and impacted files
@@ Coverage Diff @@## main #1239 +/- ##
=======================================
Coverage 90.96% 90.96% =======================================
Files 222 222 Lines 7501 7501 =======================================
Hits 6823 6823 Misses 678 678

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

@technically-tracy
technically-tracy merged commit 42af666 into mainJan 30, 2025
@technically-tracy
technically-tracy deleted the tracy-assistant-structure-py branch January 30, 2025 15:13
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@lukegalbraithrusselllukegalbraithrusselllukegalbraithrussell left review comments
@WilliamBergaminWilliamBergaminWilliamBergamin approved these changes
+1 more reviewer
@seratchseratchseratch approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

docsImprovements or additions to documentation

Projects

None yet

Milestone

1.23.0

Development

Successfully merging this pull request may close these issues.

4 participants

@technically-tracy@WilliamBergamin@seratch@lukegalbraithrussell