Docs: migration guides for new package versions#2606
Docs: migration guides for new package versions#2606lukegalbraithrussell wants to merge 12 commits into
Conversation
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## v8 #2606 +/- ##
=======================================
Coverage 88.98% 88.98%
=======================================
Files 62 62
Lines 10118 10118
Branches 458 458
=======================================
Hits 9003 9003
Misses 1091 1091
Partials 24 24
Flags with carried forward coverage won't be shown. Click here to find out more. 🚀 New features to boost your workflow:
|
WilliamBergamin
left a comment
There was a problem hiding this comment.
Nice work 🥇 left a few comments mainly about remove some stuff 🙏
Maybe we should merge this branch in the v8 instead of main that was we can merge everything at the same time 🤔 at the same time there might be a delay between merging and releasing
Co-authored-by: William Bergamin <william.bergamin.coen@gmail.com> Co-authored-by: Luke Russell <31357343+lukegalbraithrussell@users.noreply.github.com>
Co-authored-by: Luke Russell <31357343+lukegalbraithrussell@users.noreply.github.com>
…node-slack-sdk into docs-package-migration
|
@WilliamBergamin addressed all your feedback! (removed the stuff that should be removed, and swapped Option A and Option B for each) |
WilliamBergamin
left a comment
There was a problem hiding this comment.
Thanks for addressing all my feedback 🙏
|
|
||
| ### We've updated the dependency on `@slack/web-api@^8` | ||
|
|
||
| This package now uses `@slack/web-api@^8` internally. If you're passing `clientOptions`, the web-api breaking changes apply there too. Check the [web-api v8 migration guide](./web-api-v8-migration.md) for details. |
There was a problem hiding this comment.
Is the link to [web-api v8 migration guide](./web-api-v8-migration.md) valid?
Summary
previews:
Requirements