Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #2676 +/- ##
=======================================
Coverage 93.40% 93.40%
=======================================
Files 37 37
Lines 7613 7613
Branches 668 668
=======================================
Hits 7111 7111
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.
@haleychaas LGTM! And this template is nice to highlight here!
I'd be open to follow up that hints at which AI features are shown but also I understand this to be referenced elsewhere and changing perhaps frequent - no blockers I find! 🚢 💨
| :::danger[Keep your tokens and signing secret secure] | ||
|
|
||
| At a minimum, you should avoid checking tokens and signing secrets into public version control, and you should access them via environment variables as shown above. Check out the API docs for more [app security best practices](/authentication/best-practices-for-security). | ||
| At a minimum, you should avoid checking tokens and signing secrets into public version control, and you should access them via environment variables as shown above. Check out the guide to [app security best practices](/authentication/best-practices-for-security) for more insights. |
There was a problem hiding this comment.
📚 praise: I'm a fan of these wording!
docs/english/getting-started.md
Outdated
|
|
||
| On these pages you're free to make changes such as updating your app icon, configuring app features, and perhaps even distributing your app! | ||
|
|
||
| ## Add AI features {#ai-features} |
There was a problem hiding this comment.
| ## Add AI features {#ai-features} | |
| ## Adding AI features {#ai-features} |
👾 suggestion(non-blocking): To gerunding these headers reads nice to me as well-
| Get started with the agent template: | ||
|
|
||
| ```sh | ||
| $ git clone https://github.com/slack-samples/bolt-js-assistant-template ai-app |
There was a problem hiding this comment.
📝 note: I believe this link might change soon - we'll want to follow up on this but IIRC we have note of this elsewhere?
There was a problem hiding this comment.
good to know! I think we reference it elsewhere too, so we can just do a big find-and-replace when the time comes
docs/english/getting-started.md
Outdated
| ``` | ||
|
|
||
| </TabItem> | ||
| <TabItem value="terminaL" label="Terminal"> |
There was a problem hiding this comment.
| <TabItem value="terminaL" label="Terminal"> | |
| <TabItem value="terminal" label="Terminal"> |
👁️🗨️ suggestion: This selects well still for me but I am unsure if this is case sensitive?
2 participants
Summary
Per a request from leadership to add AI to the quickstart, I added a section here. Also added a bullet under "Next steps" in the Building an App guide. Also a cleanup task: removed mention of "API docs" since these docs are now in the API docs 😉
You can preview the changes here (on the Bolt JS tab): https://docs-slack-d-docs-haley-1wpghj.herokuapp.com/quickstart
Requirements (place an
xin each[ ])