Skip to content

chore: improve README intro#159

Merged
pranavjain97 merged 1 commit into
masterfrom
WP-6770-update-readme
Nov 24, 2025
Merged

chore: improve README intro#159
pranavjain97 merged 1 commit into
masterfrom
WP-6770-update-readme

Conversation

@pranavjain97
Copy link
Copy Markdown
Contributor

Ticket: WP-6770

Comment thread README.md Outdated
Comment thread README.md Outdated
- **KMS API Implementation** - Advanced Wallet Manager requires a KMS API for secure key operations. For example:
- [AWS HSM Implementation](./demo-kms-script/aws-interface.md)
- [Dinamo HSM Implementation](./demo-kms-script/dinamo-interface.md)
- **KMS API Implementation** - You must implement the KMS API interface to connect your KMS/HSM to the Advanced Wallet Manager. The interface specification is provided by BitGo. Reference implementations are available to guide your integration:
Copy link
Copy Markdown
Contributor

@vmccarty vmccarty Nov 24, 2025

Choose a reason for hiding this comment

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

Suggested change
- **KMS API Implementation** - You must implement the KMS API interface to connect your KMS/HSM to the Advanced Wallet Manager. The interface specification is provided by BitGo. Reference implementations are available to guide your integration:
- **KMS API Implementation** - You must implement the KMS API interface to connect your KMS/HSM to the Advanced Wallet Manager. BitGo provides a specification for the interface and the following example implementations:

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Where does BitGo provide the spec?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

good catch, i thought we already included it in the repo but we didn't
adding it separately and will link that here 👍🏼

Comment thread README.md Outdated
Copy link
Copy Markdown
Contributor

@vmccarty vmccarty left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@pranavjain97 pranavjain97 merged commit 5cd9750 into master Nov 24, 2025
8 checks passed
@pranavjain97 pranavjain97 deleted the WP-6770-update-readme branch November 24, 2025 22:16
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