Skip to content
This repository has been archived by the owner on Sep 28, 2022. It is now read-only.

explicitly add value property to tx options #189

Merged
merged 2 commits into from
Nov 21, 2019
Merged
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 5 additions & 0 deletions packages/dai/src/eth/TransactionManager.js
Original file line number Diff line number Diff line change
Expand Up @@ -47,6 +47,11 @@ export default class TransactionManager extends PublicService {
businessObject = options.businessObject;
delete options.businessObject;
}

// some subproviders require a value key included with the Tx
if (!options.value) {
levity marked this conversation as resolved.
Show resolved Hide resolved
options.value = 0;
}
} else {
options = {};
}
Expand Down