Skip to content

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
@salesforce-cla
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.

@codecov
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 (f7915fb) to head (af50137).

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #111   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           36        36           
  Lines         1133      1196   +63     
  Branches        17        27   +10     
=========================================
+ Hits          1133      1196   +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?

@lynfos
Copy link

lynfos commented Mar 5, 2025

bump this would seem to have some urgency since the files.upload API will stop functioning on March 11, 2025, less than one week away!

@salesforce-cla
Copy link

salesforce-cla bot commented Mar 5, 2025

Thanks for the contribution! Unfortunately we can't verify the commit author(s): Filip Maj <f***@s***.com>. One possible solution is to add that email to your GitHub account. Alternatively you can change your commits to another email and force push the change. After getting your commits associated with your GitHub account, sign the Salesforce Inc. Contributor License Agreement and this Pull Request will be revalidated.

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

7 participants