aws-amplify / amplify-ui

Amplify UI is a collection of accessible, themeable, performant React (and more!) components that can connect directly to the cloud.
https://ui.docs.amplify.aws
Apache License 2.0
919 stars 297 forks source link

fix (Storage/FileUploader): FileUploader does not upload processed file contents in certain scenarios #6050

Closed choyky closed 6 days ago

choyky commented 2 weeks ago

Description of changes

This change fix an issue in which FileUploader does not upload the processed/modified file contents as advertised in documentation.

In current implementation, the original input file is returned as-is when hasKeyInput is false. However, the processed file contents should be returned instead regardless of the condition of hasKeyInput.

This code change fixes that and returns processed/modified file contents instead of the original.

Issue #, if available

https://github.com/aws-amplify/amplify-ui/issues/6049

Description of how you validated changes

Checklist

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

changeset-bot[bot] commented 2 weeks ago

🦋 Changeset detected

Latest commit: 35298ada358779a090b9d0780f5023587ffdd9d7

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package | Name | Type | | ----------------------------- | ----- | | @aws-amplify/ui-react-storage | Patch |

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR