-
Notifications
You must be signed in to change notification settings - Fork 662
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update docs to include files.uploadV2 #1548
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@srajiang Once the version error is fixed, all look good to me
docs/_packages/web_api.md
Outdated
@@ -448,15 +448,77 @@ retrying the API call. If you'd like to opt out of that behavior, set the `rejec | |||
--- | |||
|
|||
### Upload a file | |||
As of @slack/web-api v2.8.0, we have introduced a modified way to upload files. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As of @slack/web-api v2.8.0, we have introduced a modified way to upload files. | |
As of @slack/web-api v6.8.0, we have introduced a modified way to upload files. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ooh, thanks for the catch! I've fixed it.
77e1150
to
b105e71
Compare
Thanks, this is now perfect ✅ Let's merge this PR once we ship the version! |
6.8.0 has been released: https://github.com/slackapi/node-slack-sdk/releases/tag/%40slack%2Fweb-api%406.8.0 will merge this PR in now! |
Summary
Adds docs for files.uploadV2 wrapper method.
Requirements (place an
x
in each[ ]
)