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

Add Token image from URL #4916

Merged
merged 1 commit into from
Mar 15, 2017
Merged

Add Token image from URL #4916

merged 1 commit into from
Mar 15, 2017

Conversation

ngotchac
Copy link
Contributor

No need to go the the GHH dapp for setting a Token icon: hash the given URL content and post the GHH transaction for registering it directly from the Token Reg dapp

@ngotchac ngotchac added A0-pleasereview 🤓 Pull request needs code review. M8-dapp 💎 Decentralized applications. labels Mar 15, 2017
@jacogr jacogr added A8-looksgood 🦄 Pull request is reviewed well. and removed A0-pleasereview 🤓 Pull request needs code review. labels Mar 15, 2017
@gavofyork gavofyork merged commit a25d935 into master Mar 15, 2017
@gavofyork gavofyork deleted the ng-tokenreg-enhanced branch March 15, 2017 15:49
@gavofyork
Copy link
Contributor

(generic component for this would be nice ;)

@ngotchac
Copy link
Contributor Author

100% agreed ! However we don't share component betweens dapps yet.

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