Skip to content
This repository has been archived by the owner on Nov 6, 2020. It is now read-only.

Fix method decoding #4845

Merged
merged 2 commits into from Mar 9, 2017
Merged

Fix method decoding #4845

merged 2 commits into from Mar 9, 2017

Conversation

ngotchac
Copy link
Contributor

@ngotchac ngotchac commented Mar 9, 2017

When deploying a contract, the MethodDecoding in the Signer for the pending request wasn't working...

@ngotchac ngotchac added A0-pleasereview 🤓 Pull request needs code review. M7-ui labels Mar 9, 2017
@jacogr jacogr added A8-looksgood 🦄 Pull request is reviewed well. and removed A0-pleasereview 🤓 Pull request needs code review. labels Mar 9, 2017
@jacogr jacogr merged commit 3bb66d0 into master Mar 9, 2017
@jacogr jacogr deleted the ng-fix-method-decoding branch March 9, 2017 18:56
jacogr pushed a commit that referenced this pull request Mar 10, 2017
* Fix contract deployment method decoding in Signer

* Linting
@jacogr jacogr mentioned this pull request Mar 10, 2017
arkpar pushed a commit that referenced this pull request Mar 10, 2017
* Added React Hot Reload to dapps + TokenDeplpoy fix (#4846)

* Fix method decoding (#4845)

* Fix contract deployment method decoding in Signer

* Linting

* Fix TxViewer when no `to` (contract deployment) (#4847)

* Added React Hot Reload to dapps + TokenDeplpoy fix

* Fixes to the LocalTx dapp

* Don't send the nonce for mined transactions

* Don't encode empty to values for options

* Pull steps from actual available steps (#4848)

* Wait for the value to have changed in the input (#4844)

* Backport Regsirty changes from #4589

* Test fixes for #4589
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A8-looksgood 🦄 Pull request is reviewed well.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants