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

Add React Hot Reload to dapps + TokenDeploy fix #4846

Merged
merged 1 commit into from
Mar 9, 2017
Merged

Conversation

ngotchac
Copy link
Contributor

@ngotchac ngotchac commented Mar 9, 2017

The TokenDeploy dapps wasn't working anymore...
This adds React Hot Reload to dapps.

NB : dappreg has been omitted since it's done in #4589

@ngotchac ngotchac added A0-pleasereview 🤓 Pull request needs code review. M8-dapp 💎 Decentralized applications. 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
Copy link
Contributor

jacogr commented Mar 9, 2017

Yes, saw - going through the dappreg PR, just playing some.

@jacogr jacogr merged commit 3478c16 into master Mar 9, 2017
@jacogr jacogr deleted the ng-fix-localtx branch March 9, 2017 18:55
@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. M8-dapp 💎 Decentralized applications.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants