Skip to content
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

App Center: upgrade app #714

Merged
merged 3 commits into from
Apr 17, 2019
Merged

App Center: upgrade app #714

merged 3 commits into from
Apr 17, 2019

Conversation

sohkai
Copy link
Contributor

@sohkai sohkai commented Apr 16, 2019

The radspec description is too technical for normal users right now, and will require an @aragon/wrapper upgrade to prettify it (see aragon/aragon.js#284 for how it renders).

Screen Shot 2019-04-17 at 12 35 33 AM

margin-right: 4px;
`}
>
<LocalIdentityBadge
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ideally this should the be AppBadge cc @bpierre.

@sohkai sohkai force-pushed the app-center-upgrade branch from a499d89 to 66f9b13 Compare April 16, 2019 22:35
@bpierre
Copy link
Contributor

bpierre commented Apr 17, 2019

I increased a bit the line height for now:

/cc @dizzypaty

@sohkai sohkai merged commit 95049f4 into master Apr 17, 2019
@sohkai sohkai deleted the app-center-upgrade branch April 17, 2019 14:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants