Skip to content

feat: access key custom attributes CRU#645

Merged
aviadl merged 3 commits intomainfrom
access-key-custom-attributes
Jan 18, 2026
Merged

feat: access key custom attributes CRU#645
aviadl merged 3 commits intomainfrom
access-key-custom-attributes

Conversation

@aviadl
Copy link
Member

@aviadl aviadl commented Jan 17, 2026

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adds support for custom attributes to access key Create, Read (search), and Update operations, related to issue #13177.

Changes:

  • Added customAttributes parameter to create, searchAll, and update methods in the access key management API
  • Added boundUserId and creatingUser search filter parameters to searchAll method
  • Updated test coverage to verify the new parameters are properly passed through to the API calls

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

File Description
lib/management/accesskey.ts Adds customAttributes parameter to create, searchAll, and update methods; adds boundUserId and creatingUser filters to searchAll
lib/management/accesskey.test.ts Updates test cases to verify new parameters are correctly passed to API endpoints
README.md Updates create method example to demonstrate customAttributes usage

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

aviadl and others added 2 commits January 17, 2026 21:45
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@aviadl aviadl merged commit 145d363 into main Jan 18, 2026
27 checks passed
@aviadl aviadl deleted the access-key-custom-attributes branch January 18, 2026 04:56
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