Skip to content

Configure UserGroup & UserInGroup resources from the cognitoidp group - #266

Merged
ulucinar merged 1 commit into
crossplane-contrib:mainfrom
ulucinar:fix-aws-25
Dec 13, 2022
Merged

Configure UserGroup & UserInGroup resources from the cognitoidp group#266
ulucinar merged 1 commit into
crossplane-contrib:mainfrom
ulucinar:fix-aws-25

Conversation

@ulucinar

@ulucinar ulucinar commented Dec 13, 2022

Copy link
Copy Markdown
Collaborator

Description of your changes

Fixes #25

I have:

  • Run make reviewable test to ensure this PR is ready for review.

How has this code been tested

@ulucinar
ulucinar marked this pull request as draft December 13, 2022 09:50
@ulucinar

Copy link
Copy Markdown
Collaborator Author

/test-examples="examples/cognitoidp/usergroup.yaml"

@ulucinar

Copy link
Copy Markdown
Collaborator Author

/test-examples="examples/cognitoidp/useringroup.yaml"

- Fixes #25

Signed-off-by: Alper Rifat Ulucinar <ulucinar@users.noreply.github.com>
@ulucinar

Copy link
Copy Markdown
Collaborator Author

/test-examples="examples/cognitoidp/useringroup.yaml"

@ulucinar
ulucinar marked this pull request as ready for review December 13, 2022 10:26
@ulucinar

Copy link
Copy Markdown
Collaborator Author

/test-examples="examples/cognitoidp/useringroup.yaml"

@sergenyalcin sergenyalcin left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thanks @ulucinar LGTM!

@ulucinar
ulucinar merged commit 043d71e into crossplane-contrib:main Dec 13, 2022
@ulucinar
ulucinar deleted the fix-aws-25 branch December 13, 2022 12:28
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.

aws.cognitoidp: usergroup and useringroup are not getting deleted complaining on non-exsisting User pool

2 participants