-
Notifications
You must be signed in to change notification settings - Fork 200
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
feat(storage): update Storage APIs with StoragePath parameter #3585
Conversation
* feat(storage): refactor storage remove api by including path * updated tests
Codecov ReportAttention: Patch coverage is
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## main #3585 +/- ##
==========================================
+ Coverage 68.47% 71.05% +2.57%
==========================================
Files 1077 1012 -65
Lines 36282 33463 -2819
==========================================
- Hits 24844 23777 -1067
+ Misses 11438 9686 -1752
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Issue #
Description
StoragePath
StoragePath
parameter. The following APIs are udpated: getURL, list, remove, downloadData, downloadFile, uploadData, uploadFileAll code changes have been previously reviewed and approved:
path
#3573path
#3580General Checklist
Given When Then
inline code documentation and are named accordinglytestThing_condition_expectation()
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.