Skip to content

feat(sdk-core): added OFC BitGo signing on wallet and coins object#8803

Merged
alextse-bg merged 1 commit into
masterfrom
WCN-217
May 20, 2026
Merged

feat(sdk-core): added OFC BitGo signing on wallet and coins object#8803
alextse-bg merged 1 commit into
masterfrom
WCN-217

Conversation

@alextse-bg
Copy link
Copy Markdown
Contributor

allow wallet and coins object to sign using the BitGo key if the passphrase is not provided during signing

Ticket: WCN-217-2

@linear-code
Copy link
Copy Markdown

linear-code Bot commented May 19, 2026

WCN-217

@alextse-bg alextse-bg force-pushed the WCN-217 branch 2 times, most recently from a09e279 to 16bcfef Compare May 19, 2026 16:57
@alextse-bg alextse-bg marked this pull request as ready for review May 19, 2026 17:41
@alextse-bg alextse-bg requested review from a team as code owners May 19, 2026 17:41
@alextse-bg alextse-bg requested a review from ranga-r2 May 19, 2026 17:41
Comment thread modules/sdk-core/src/coins/ofcToken.ts Outdated
Comment thread modules/sdk-core/src/bitgo/trading/tradingAccount.ts Outdated
Comment thread modules/sdk-core/src/coins/ofcToken.ts Outdated
Comment thread modules/sdk-core/src/coins/ofcToken.ts Outdated
@alextse-bg alextse-bg marked this pull request as draft May 19, 2026 18:32
Comment thread modules/sdk-core/src/coins/ofcToken.ts Outdated
allow wallet and coins object to sign using the BitGo key
if the passphrase is not provided during signing

Ticket: WCN-217-2
@alextse-bg alextse-bg marked this pull request as ready for review May 19, 2026 20:25
Copy link
Copy Markdown
Contributor

@mohammadalfaiyazbitgo mohammadalfaiyazbitgo left a comment

Choose a reason for hiding this comment

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

Blocking on the walletPassphrase gap in the wallet.ts OFC path — see inline comment. Also a minor error message inconsistency in ofcToken.ts.

Comment thread modules/sdk-core/src/bitgo/wallet/wallet.ts
Comment thread modules/sdk-core/src/coins/ofcToken.ts
@alextse-bg alextse-bg merged commit f450bda into master May 20, 2026
27 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.

2 participants