Skip to content
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

[FEATURE] Add files.uploadV2 like the other Slack SDKs have #89

Open
filmaj opened this issue Dec 7, 2023 · 1 comment
Open

[FEATURE] Add files.uploadV2 like the other Slack SDKs have #89

filmaj opened this issue Dec 7, 2023 · 1 comment
Labels
enhancement New feature or request help wanted Extra attention is needed semver:minor requires a minor version number bump

Comments

@filmaj
Copy link
Contributor

filmaj commented Dec 7, 2023

See https://slack.dev/node-slack-sdk/web-api#upload-a-file for docs on how this method works from a user perspective.

Here is the code from the node-slack-sdk for this method: https://github.com/slackapi/node-slack-sdk/blob/main/packages/web-api/src/file-upload.ts

@seratch has example code for deno-slack-sdk-based apps that use the above approach, more or less, that we could probably lift from: https://github.com/seratch/slack-timesheet/blob/412d9899c839fe24531a097ab0f1f68eb0513d2a/functions/internals/reports.ts#L549-L590

@filmaj filmaj added enhancement New feature or request help wanted Extra attention is needed semver:minor requires a minor version number bump labels Dec 7, 2023
@taiga533 taiga533 mentioned this issue Dec 15, 2023
3 tasks
@taiga533
Copy link

Hi @filmaj
I implemented fileUploadV2 method in base-client and created merge request.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request help wanted Extra attention is needed semver:minor requires a minor version number bump
Projects
None yet
Development

No branches or pull requests

2 participants