Skip to content
This repository has been archived by the owner on Jul 9, 2021. It is now read-only.

Add batchExecuteTransactions #1793

Merged
merged 13 commits into from
Apr 29, 2019
Merged

Add batchExecuteTransactions #1793

merged 13 commits into from
Apr 29, 2019

Conversation

abandeali1
Copy link
Member

@abandeali1 abandeali1 commented Apr 25, 2019

Description

  • Adds a bytes return value to executeTransaction. It will now return whatever data was returned by the underlying function call
  • Adds a batchExecuteTransactions function
  • Fixes preSign to be compatible with executeTransaction
  • Refactors the transaction tests (this is where most of the code actually is)

Testing instructions

Types of changes

Checklist:

  • Prefix PR title with [WIP] if necessary.
  • Add tests to cover changes as needed.
  • Update documentation as needed.
  • Add new entries to the relevant CHANGELOG.jsons.

@abandeali1 abandeali1 requested a review from hysz as a code owner April 25, 2019 23:11
@abandeali1 abandeali1 changed the title Feat/3.0/transactions [WIP] Add batchExecuteTransactions Apr 25, 2019
@coveralls
Copy link

coveralls commented Apr 25, 2019

Coverage Status

Coverage remained the same at 84.595% when pulling fd8a8b9 on feat/3.0/transactions into 02d8ef1 on 3.0.

@abandeali1 abandeali1 changed the title [WIP] Add batchExecuteTransactions Add batchExecuteTransactions Apr 26, 2019
Copy link
Contributor

@dorothy-zbornak dorothy-zbornak left a comment

Choose a reason for hiding this comment

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

Small nits, but looks nice!


/// @dev Executes a batch of Exchange method calls in the context of signer(s).
/// @param transactions Array of 0x transactions containing salt, signerAddress, and data.
/// @param signatures Array of proofs that transactions have been signed by signer(s).
Copy link
Contributor

Choose a reason for hiding this comment

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

Missing @return natspec.

/// @param transaction 0x transaction containing salt, signerAddress, and data.
/// @param signature Proof of signer transaction by signer.
/// @param signature Proof that transaction has been signed by signer.
Copy link
Contributor

Choose a reason for hiding this comment

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

Missing @return natspec.

@abandeali1 abandeali1 merged commit 02f6cb8 into 3.0 Apr 29, 2019
@abandeali1 abandeali1 deleted the feat/3.0/transactions branch June 1, 2019 20:43
@dekz dekz added this to the v3 development milestone Jun 21, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants