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

Add files.uploadV2 #111

Draft
wants to merge 4 commits into
base: main
Choose a base branch
from
Draft

Add files.uploadV2 #111

wants to merge 4 commits into from

Conversation

filmaj
Copy link
Contributor

@filmaj filmaj commented Aug 27, 2024

⚠️ WIP!

This PR supersedes #90 and fixes #89. It builds upon previous work done by @taiga533; I intend to get it over the finish line.

TODO

Generate files.getUploadURLExternal and files.completeUploadExternal.
and add new tests to api_test.ts.
Modify the definition of the FileUploadV2 type to extend the FileUpload type.
And modify the FileUpload type so that it does not directly reference ReadableStream, ensuring npm build succeeds.
@filmaj filmaj self-assigned this Aug 27, 2024
Copy link

Thanks for the contribution! Before we can merge this, we need @racTaiga533 to sign the Salesforce Inc. Contributor License Agreement.

@filmaj filmaj mentioned this pull request Aug 27, 2024
3 tasks
@filmaj
Copy link
Contributor Author

filmaj commented Aug 27, 2024

Confirmed that we have a CLA on file for @racTaiga533 (signed 2023/12/15) - can ignore the CLA bot's complaints.

Copy link

codecov bot commented Aug 27, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 100.00%. Comparing base (3d2507e) to head (043f923).

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #111   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           36        36           
  Lines         1130      1193   +63     
  Branches        17        27   +10     
=========================================
+ Hits          1130      1193   +63     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@WilliamBergamin WilliamBergamin self-requested a review August 28, 2024 14:13
@deefactorial
Copy link

deefactorial commented Oct 6, 2024

What's the status on this PR? I need this functionality, it would be nice if it was merged and deployed.
@WilliamBergamin @filmaj

@deefactorial
Copy link

Screenshot 2024-11-07 at 11 28 11 AM
still not working, is there any way I can use this PR temporarily until it's merged?

@simhnna
Copy link

simhnna commented Dec 4, 2024

The code provided does work but I'd probably change the API a bit. Is this moving forward? Shall/May I take over?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FEATURE] Add files.uploadV2 like the other Slack SDKs have
4 participants