Skip to content

Commit

Permalink
Merge pull request #3720 from BitGo/EA-549-sdk-support-stake-many-for…
Browse files Browse the repository at this point in the history
…-sui

fix(sdk-core): make amount optional in stake options
  • Loading branch information
ewangbitgo committed Jul 13, 2023
2 parents e884193 + 19039a5 commit 5876caf
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 3 deletions.
2 changes: 0 additions & 2 deletions modules/bitgo/test/v2/unit/staking/stakingWalletCommon.ts
Original file line number Diff line number Diff line change
Expand Up @@ -81,7 +81,6 @@ describe('Staking Wallet Common', function () {
const expected = fixtures.stakingRequest([fixtures.transaction('NEW')]);
const msScope = nock(microservicesUri)
.post(`/api/staking/v1/${stakingWallet.coin}/wallets/${stakingWallet.walletId}/requests`, {
amount: '1',
clientId: 'clientId',
type: 'STAKE',
delegationRequests: [
Expand All @@ -93,7 +92,6 @@ describe('Staking Wallet Common', function () {
.reply(201, expected);

const options = {
amount: '1',
clientId: 'clientId',
delegationRequests: [
{ amount: '1', validator: '123' },
Expand Down
2 changes: 1 addition & 1 deletion modules/sdk-core/src/bitgo/staking/iStakingWallet.ts
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ export interface DelegationRequest {
}

export interface StakeOptions {
amount: string;
amount?: string;
clientId?: string;
validator?: string;
durationSeconds?: string;
Expand Down

0 comments on commit 5876caf

Please sign in to comment.