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

Dapps errors embeddable on signer #3115

Merged
merged 1 commit into from
Nov 3, 2016
Merged

Dapps errors embeddable on signer #3115

merged 1 commit into from
Nov 3, 2016

Conversation

tomusdrw
Copy link
Collaborator

@tomusdrw tomusdrw commented Nov 3, 2016

Example: When trying to load GavCoin (it's a network dapp now) and the node is still syncing the error should be displayed, but you see a white page because it's prevented from being embedded in an iframe.

@tomusdrw tomusdrw added A0-pleasereview 🤓 Pull request needs code review. M4-core ⛓ Core client code / Rust. labels Nov 3, 2016
@gavofyork gavofyork added A8-looksgood 🦄 Pull request is reviewed well. and removed A0-pleasereview 🤓 Pull request needs code review. labels Nov 3, 2016
@gavofyork gavofyork merged commit e251fd4 into master Nov 3, 2016
@gavofyork gavofyork deleted the dapps-errors branch November 3, 2016 11:22
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. M4-core ⛓ Core client code / Rust.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants