Skip to content

Add AdminLinkProviderForUser action to manageUsers ActionMap#3075

Open
javabudd wants to merge 9 commits intoaws-amplify:mainfrom
javabudd:main
Open

Add AdminLinkProviderForUser action to manageUsers ActionMap#3075
javabudd wants to merge 9 commits intoaws-amplify:mainfrom
javabudd:main

Conversation

@javabudd
Copy link
Copy Markdown

Problem

Missing AdminLinkProviderForUser action from manageUsers

#3070

Changes

Adds AdminLinkProviderForUser action to the manageUsers ActionMap

Validation

Tested this change locally and saw that AdminLinkProviderForUser was added to the policy generated by the manageUsers ActionMap

Checklist

N/A

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

@javabudd javabudd requested a review from a team as a code owner December 28, 2025 20:55
@changeset-bot
Copy link
Copy Markdown

changeset-bot bot commented Dec 28, 2025

🦋 Changeset detected

Latest commit: 0eac215

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/backend-auth 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

@javabudd
Copy link
Copy Markdown
Author

@pahud Any idea when we can expect this to be tagged and merged?

@javabudd
Copy link
Copy Markdown
Author

@pahud I'm getting loads of notifications around this pipeline failing over and over (health_checks -> dependency-review). I figured I'd let you know in case this is burning CI credits.

Copy link
Copy Markdown
Contributor

@ShadowCat567 ShadowCat567 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hello @javabudd! Thank you for the contribution (and sorry for taking so long to review it). I have a suggestion for you since I think there may be a better home for cognito-idp:AdminLinkProviderForUser in the action map than under manageUsers. Let me know what you think!

Comment thread packages/backend-auth/src/userpool_access_policy_factory.ts Outdated
@javabudd
Copy link
Copy Markdown
Author

javabudd commented Mar 7, 2026

Hello @javabudd! Thank you for the contribution (and sorry for taking so long to review it). I have a suggestion for you since I think there may be a better home for cognito-idp:AdminLinkProviderForUser in the action map than under manageUsers. Let me know what you think!

Thanks for the feedback, this has been updated.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants