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

Fix console dapp #4544

Merged
merged 2 commits into from
Feb 14, 2017
Merged

Fix console dapp #4544

merged 2 commits into from
Feb 14, 2017

Conversation

tomusdrw
Copy link
Collaborator

  • Enables linting for console.js and fixes all errors.
  • Fixes web3 RPC endpoint, adding CORS headers for signer.
  • Exposes all possible custom methods as web3 extensions.

@tomusdrw tomusdrw added A0-pleasereview 🤓 Pull request needs code review. B0-patch M8-dapp 💎 Decentralized applications. labels Feb 14, 2017
@jacogr
Copy link
Contributor

jacogr commented Feb 14, 2017

Really focussed only on the fixing part, which seems to cover the instances that were problematic.

Thanks for taking the time to do the eslint fixes as well, gave that a walk-though, much better. (The reformat closes https://github.com/ethcore/parity/issues/4157)

@jacogr jacogr added A8-looksgood 🦄 Pull request is reviewed well. and removed A0-pleasereview 🤓 Pull request needs code review. labels Feb 14, 2017
@gavofyork gavofyork merged commit d925cc0 into master Feb 14, 2017
@gavofyork gavofyork deleted the console-fixes branch February 14, 2017 21:45
tomusdrw added a commit that referenced this pull request Feb 16, 2017
* Fixing linting issues. Better support for console as secure app

* Fixing linting issues
gavofyork pushed a commit that referenced this pull request Feb 16, 2017
* Fixing evmbin compilation and added standard build. (#4561)

* Alias for personal_sendTransaction (#4554)

* Fix console dapp (#4544)

* Fixing linting issues. Better support for console as secure app

* Fixing linting issues

* Fix no data sent in TxQueue dapp (#4502)

* Fix wrong PropType req for Embedded Signer

* Fix wrong data for tx #4499
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.

3 participants