Docs: edits for "guide building an app tutorial using the slack cli" #2598
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## zimeg-docs-getting-started-cli #2598 +/- ##
===============================================================
Coverage 93.37% 93.37%
===============================================================
Files 37 37
Lines 7581 7581
Branches 667 667
===============================================================
Hits 7079 7079
Misses 497 497
Partials 5 5 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
zimeg
left a comment
zimeg
left a comment
There was a problem hiding this comment.
@lukegalbraithrussell LGTM! I'm amazed at the seemingly quick calls made to improve the writing so much 🙏 ✨
I left a few comments of thoughts and links and organization alike. Please feel free to ignore most or all.
We can also prefer edits on this branch instead of #2597 if that'd be best? 🤖
The sample app still needs to be published with some CLI changes, but for now we should keep these edits around! 🚀
| Look around, add an app icon and description, and then let's start configuring your app. 🔩 | ||
|
|
||
| #### Installing the app | ||
| #### Installing the app {#installing-the-app} |
| * set up your local environment, | ||
| * create a new Slack app, | ||
| * and enable it to listen for and respond to messages within a Slack workspace. |
|
|
||
| ::: | ||
|
|
||
| #### Prerequisites |
There was a problem hiding this comment.
🪓 Good call for fast linking ahead!
| </Tabs> | ||
|
|
||
| Your app should let you know that it's up and running. 🎉 | ||
| Your app should let you know that it's up and running. It's not actually listening for anything though. Let's change that. |
| <TabItem value="socket-mode" label="Socket Mode"> | ||
|
|
||
| ```javascript | ||
| ```javascript title="app.js" |
Co-authored-by: Michael Brooks <[email protected]>
|
☝️ Failing tests might require a rebase with The referenced sample app and CLI changes are in progress but after those land the guide will be in a good place. 🤖 |
|
ah right yes, pls merge this into your PR whenever it is convenient for you! |
|
@lukegalbraithrussell Will do, of course with all credit of these changes and a huge thanks to you 🙏 ✨ |
Summary
This provides some edits for #2597
Requirements (place an
xin each[ ])