Skip to content

Commit

Permalink
Merge pull request #103 from lidofinance/feature/si-1210-fix-stake-po…
Browse files Browse the repository at this point in the history
…pulate-tx

[Fix] stake populate tx
  • Loading branch information
Jeday authored Jan 26, 2024
2 parents ecba987 + bddde3c commit e6a81d3
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 5 deletions.
1 change: 1 addition & 0 deletions packages/sdk/CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,7 @@

- better multisig behavior for transactions
- Simulate methods now have correct return types
- `stakeEthPopulateTx` not does not calculate `gasLimit` which prevented usage when stake limit is reached

## Playground

Expand Down
5 changes: 0 additions & 5 deletions packages/sdk/src/stake/stake.ts
Original file line number Diff line number Diff line change
Expand Up @@ -240,15 +240,10 @@ export class LidoSDKStake extends LidoSDKModule {
const { referralAddress, value, account } = await this.parseProps(props);
const data = this.stakeEthEncodeData({ referralAddress });
const address = await this.contractAddressStETH();
const gas = await this.submitGasLimit(value, referralAddress, {
account,
chain: this.core.chain,
});
return {
to: address,
from: account.address,
value,
gas,
data,
};
}
Expand Down

0 comments on commit e6a81d3

Please sign in to comment.