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

Load network apps manifests as contentHash (no coding) #3235

Merged
merged 20 commits into from
Nov 7, 2016

Conversation

jacogr
Copy link
Contributor

@jacogr jacogr commented Nov 7, 2016

  • add meta key to dappRegistry - MANIFEST
  • load manifest via contentHash via this key
  • deployed on Mordent for Gavcoin

@jacogr jacogr added A0-pleasereview 🤓 Pull request needs code review. M6-ui labels Nov 7, 2016
@jacogr jacogr added B0-patch A8-looksgood 🦄 Pull request is reviewed well. and removed A0-pleasereview 🤓 Pull request needs code review. labels Nov 7, 2016
@arkpar arkpar merged commit 6ab6c07 into master Nov 7, 2016
@arkpar arkpar deleted the jg-network-manifest branch November 7, 2016 17:08
@jacogr
Copy link
Contributor Author

jacogr commented Nov 7, 2016

arkpar pushed a commit that referenced this pull request Nov 7, 2016
* Add mobx

* Use mobx store for dapps

* Cleanup hidden reads

* Remove (now) unused hidden.js

* _ denotes internal functions

* s/visibleApps/visible/

* AddDapps now use the mobx store as well

* Move modalOpen state to store

* Simplify

* Complete master merge

* Remove extra indirection

* Remove unneeded check

* Readability improvements

* Remove final debug info

* Load network manifests from the network

* Swallow manifest errors

* introduce fetchManifest
arkpar added a commit that referenced this pull request Nov 7, 2016
* Fixed some typos (#3236)

* Add store for dapps state (#3211)

* Add mobx

* Use mobx store for dapps

* Cleanup hidden reads

* Remove (now) unused hidden.js

* _ denotes internal functions

* s/visibleApps/visible/

* AddDapps now use the mobx store as well

* Move modalOpen state to store

* Simplify

* Complete master merge

* Remove extra indirection

* Remove unneeded check

* Readability improvements

* Remove final debug info

* Load network apps manifests as contentHash (no coding) (#3235)

* Add mobx

* Use mobx store for dapps

* Cleanup hidden reads

* Remove (now) unused hidden.js

* _ denotes internal functions

* s/visibleApps/visible/

* AddDapps now use the mobx store as well

* Move modalOpen state to store

* Simplify

* Complete master merge

* Remove extra indirection

* Remove unneeded check

* Readability improvements

* Remove final debug info

* Load network manifests from the network

* Swallow manifest errors

* introduce fetchManifest

* Rename cli and config options signer->ui (#3232)

* rename options signer->ui

* rename config signer->ui
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.

2 participants