Skip to content

Conversation

@alextse-bg
Copy link
Contributor

@alextse-bg alextse-bg commented Nov 20, 2025

… wallet

added pub as a part of the request to update keychain password for ofc multi-user-key wallet

Ticket: WP-6901

@alextse-bg alextse-bg marked this pull request as ready for review November 20, 2025 19:15
@alextse-bg alextse-bg requested a review from a team as a code owner November 20, 2025 19:15
Copy link
Contributor

@Taseen08 Taseen08 left a comment

Choose a reason for hiding this comment

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

One comment re: the encryptedPrivKeys.json. Looks good otherwise

@alextse-bg alextse-bg force-pushed the WP-6901 branch 4 times, most recently from d01fafc to b0db0e1 Compare November 21, 2025 14:54
Taseen08
Taseen08 previously approved these changes Nov 21, 2025
Copy link
Contributor

@Taseen08 Taseen08 left a comment

Choose a reason for hiding this comment

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

lgtm

zahin-mohammad
zahin-mohammad previously approved these changes Nov 24, 2025
@adityasoni-bitgo
Copy link
Contributor

CI seems to be failing here

… wallet

added pub as a part of the request to update keychain password for
ofc multi-user-key wallet, since mutliple pub-prv key pairs can exist
in the same keychain document for those wallet.

Ticket: WP-6901
@zahin-mohammad zahin-mohammad merged commit b2f6269 into master Nov 26, 2025
14 checks passed
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.

5 participants