Add AdminLinkProviderForUser action to manageUsers ActionMap#3075
Add AdminLinkProviderForUser action to manageUsers ActionMap#3075javabudd wants to merge 9 commits intoaws-amplify:mainfrom
Conversation
🦋 Changeset detectedLatest commit: 0eac215 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
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 |
|
@pahud Any idea when we can expect this to be tagged and merged? |
|
@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. |
ShadowCat567
left a comment
There was a problem hiding this comment.
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. |
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.