-
Notifications
You must be signed in to change notification settings - Fork 5.9k
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
add taraxa chains to chainlist #1805
Conversation
_data/icons/taraxa.json
Outdated
"width": 512, | ||
"height": 512, |
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.
size seems to be wrong
identify QmQhdktNyBeXmCaVuQpi1B4yXheSUKrJA17L4wpECKzG5D 19:30:11
QmQhdktNyBeXmCaVuQpi1B4yXheSUKrJA17L4wpECKzG5D PNG 310x310 310x310+0+0 8-bit sRGB 28094B 0.000u 0:00.000
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.
Hmm weird, our designer named it 512. I'm addressing it now:)
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.
@ligi Corrected to310x310
. Thanks for the review!
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.
the explorer does not really seem to support EIP3091 when it comes to blocks
@ligi not in the current version but we're weeks away from releasing the next version, which, besides having additional features, supports EIP-3091 too. However, in this case I can remove it until. |
@ligi updated without EIP-3091 |
Co-authored-by: ligi <ligi@ligi.de>
Co-authored-by: ligi <ligi@ligi.de>
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.
short name must be unique - but found TARA
more than once
@ligi fixed |
Co-authored-by: ligi <ligi@ligi.de>
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.
Please run the prettier to format your json according to the style defined here
e.g. run npx prettier --write _data/*/*.json
@ligi formatted per your request |
No description provided.