Skip to content

Conversation

@briskt
Copy link
Contributor

@briskt briskt commented Nov 3, 2025

Added

  • Include all ApiKey properties in the response to ActivateApiKey.

@briskt briskt requested review from a team as code owners November 3, 2025 07:52
@briskt briskt requested review from ethancanne, forevermatt, hobbitronics and jason-jackson and removed request for a team November 3, 2025 07:52
@sonarqubecloud
Copy link

sonarqubecloud bot commented Nov 3, 2025

Copy link
Contributor

@forevermatt forevermatt left a comment

Choose a reason for hiding this comment

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

I'm curious what the benefits of this change are. I assume it eases testing or automation somehow.

@briskt
Copy link
Contributor Author

briskt commented Nov 4, 2025

I'm curious what the benefits of this change are. I assume it eases testing or automation somehow.

Not really. It just seemed non-RESTful to omit resource data.

@briskt briskt merged commit bddd35a into develop Nov 4, 2025
6 checks passed
@briskt briskt deleted the activate-response branch November 4, 2025 06:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants