Skip to content

Comments

feat: Add account get/list tools#13

Merged
egorpavlikhin merged 3 commits intomainfrom
egorp/accounts-tools
Sep 18, 2025
Merged

feat: Add account get/list tools#13
egorpavlikhin merged 3 commits intomainfrom
egorp/accounts-tools

Conversation

@egorpavlikhin
Copy link
Contributor

Same as the certificates PR this one adds get/list tools for Accounts by querying the public endpoints and mapping the resources, excluding sensitive values.

The types were not available in the typescript client so I copied them over from the frontend code.

tenantedDeploymentParticipation: account.TenantedDeploymentParticipation,
};

// Add type-specific properties based on AccountType
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This feels a bit pointless since most properties are simply translated to camelCase, the only exception are excluded sensitive values. Happy to revisit the mapping story later on.

Copy link
Contributor

@andrewabest andrewabest left a comment

Choose a reason for hiding this comment

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

All looks good 👍

@egorpavlikhin egorpavlikhin merged commit 237cff9 into main Sep 18, 2025
@egorpavlikhin egorpavlikhin deleted the egorp/accounts-tools branch September 18, 2025 06:34
egorpavlikhin pushed a commit that referenced this pull request Sep 19, 2025
🤖 I have created a release *beep* *boop*
---


##
[0.2.0](v0.1.2...v0.2.0)
(2025-09-19)


### Features

* Add account get/list tools
([#13](#13))
([237cff9](237cff9))
* Add get/list certificates tools
([#9](#9))
([5119956](5119956))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
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.

2 participants