-
Notifications
You must be signed in to change notification settings - Fork 54
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Implement dynamic url for etherscan links and use IdentityBadge from @aragon/ui #536
Conversation
d7b7e3c
to
c9cd79b
Compare
Pull Request Test Coverage Report for Build 1605
💛 - Coveralls |
Can you pull from dev to resolve the coveralls issue? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, the badges look a little weird on the next line I'd probably change them to be inline but the way you've got it syncs with the design. Maybe something to revisit? Also since we're only testing in on localhost we should make sure it works on Rinkeby when we do that but it looks like it should work idk why it wouldn't.
Solves #182 and #317