Skip to content

Conversation

@Simone319
Copy link
Contributor

Description of changes

This change deduplicates storage browser locations when the same path appears with different permission levels. Previously, if a user had multiple access grants to the same location with varying permissions (e.g., folder/* with both READ and READ/WRITE), duplicate entries would appear. Now, the handler consolidates these into a single location entry with the broadest permission level, simplifying the location list and providing a clearer view of access rights.

Examples:

  • READ + READWRITE → Keep READWRITE (superset)
  • READ + READ → Keep first (identical)
  • READ + WRITE → Keep both (not superset, need separate locations)

Issue #, if available

Description of how you validated changes

  • unit tests
  • tested with local sample app

Checklist

  • Have read the Pull Request Guidelines
  • PR description included
  • yarn test passes and tests are updated/added
  • PR title and commit messages follow conventional commit syntax
  • If this change should result in a version bump, changeset added (This can be done after creating the PR.) This does not apply to changes made to docs, e2e, examples, or other private packages.

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

@Simone319 Simone319 added the run-tests Adding this label will trigger tests to run label Dec 9, 2025
@Simone319 Simone319 requested a review from a team as a code owner December 9, 2025 17:16
@changeset-bot
Copy link

changeset-bot bot commented Dec 9, 2025

🦋 Changeset detected

Latest commit: 57e7e3c

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

This PR includes changesets to release 2 packages
Name Type
@aws-amplify/ui-react-storage Minor
@aws-amplify/ui-test-utils 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

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

Labels

run-tests Adding this label will trigger tests to run

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants