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

Make locally installed apps available again (Fixes #2771) #2808

Merged
merged 1 commit into from
Oct 22, 2016
Merged

Make locally installed apps available again (Fixes #2771) #2808

merged 1 commit into from
Oct 22, 2016

Conversation

jacogr
Copy link
Contributor

@jacogr jacogr commented Oct 22, 2016

No description provided.

@jacogr jacogr added A0-pleasereview 🤓 Pull request needs code review. M5-ui labels Oct 22, 2016
@coveralls
Copy link

Coverage Status

Coverage increased (+0.007%) to 86.244% when pulling c5ca8ef on jg-dapp-fixes into 479657b on master.

@jacogr jacogr changed the title Make local apps available (Fixes #2771) Make local apps available (Fixes #2771, Closes #2789) Oct 22, 2016
@jacogr jacogr changed the title Make local apps available (Fixes #2771, Closes #2789) Make local apps available (Fixes #2771) Oct 22, 2016
@jacogr jacogr changed the title Make local apps available (Fixes #2771) Make locally installed apps available again (Fixes #2771) Oct 22, 2016
@jacogr
Copy link
Contributor Author

jacogr commented Oct 22, 2016

Merging to address https://github.com/ethcore/parity/pull/2793 in one go

@jacogr jacogr merged commit a1266fc into master Oct 22, 2016
@jacogr jacogr deleted the jg-dapp-fixes branch October 22, 2016 13:49
@derhuerst
Copy link
Contributor

Looks good, although the diff is almost unreadable.

@derhuerst
Copy link
Contributor

In #2721, I added the hash/id of the dapp to the UI, which you said is needed. Is that still up to date?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
A0-pleasereview 🤓 Pull request needs code review.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@jacogr @coveralls @derhuerst