-
Notifications
You must be signed in to change notification settings - Fork 42
feat(IAM Identity): adding Identity Preference API #284
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Please amend your commit with the Edit: also, there's a linter error present. In your local sandbox, please run Also, assuming that is all clean please post evidence that the integration tests and examples code runs clean. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code changes look good, but just need to address the items I mentioned in my other comments.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
whoops, inadvertently clicked Approve :)
8293a47
to
547d454
Compare
Signed-off-by: poorani <[email protected]>
547d454
to
bb9c8bd
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've pushed changes to fix the linter errors.
# [0.61.0](v0.60.0...v0.61.0) (2025-03-07) ### Features * **IAM Identity:** add Identity Preference API ([#284](#284)) ([93862bd](93862bd))
🎉 This PR is included in version 0.61.0 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
PR summary
PR Checklist
Please make sure that your PR fulfills the following requirements:
Current vs new behavior
Does this PR introduce a breaking change?
Other information