Closed saadahmsiddiqui closed 1 month ago
Overall correct, though on the contracts side optionalGas if included must be accompanied by the optionalMessage, otherwise contract call would not happen. So allowing users to only include gas might create a false impression that it affects anything.
Implementation details
With the introduction of the new
ERC20Handler
, the SDK needs to be updated to handle the enhanced functionality that supports both ERC20 token transfers and optional contract calls in a single transaction. New ERC20Handler PRERC20Handler
by allowing it to handle additional data that encodes contract calls alongside ERC20 transfers.Closes: #493
Testing details
Acceptance Criteria
ERC20Handler
, allowing for both ERC20 transfers and optional contract calls within the same transaction.