-
Notifications
You must be signed in to change notification settings - Fork 1.5k
Updated Neutron logo #6605
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
base: master
Are you sure you want to change the base?
Updated Neutron logo #6605
Conversation
Updated logos for NTRN and Neutron.
There is apparent duplication of the (chain) logo (neutron and neutron-raw) are now visually identical, so we should really only have one set. Please delete one set and remove and image URL references the refer to the set being deleted. |
Removed the images and added some more information regarding the maxbtc token. |
Changes done @JeremyParish69 |
Please ensure that any images added or removed also have their references also adjusted. (neutron-raw is still being referenced in the chain.json, but doesn't exist anymore). Also, the latest set of images (ntrn) was uploaded to the root directory instead of within the /neutron/images/ directory; please relocate those. |
Some Chain Images do not exist! |
Removed unused image entries from chain.json.
"display": "maxBTC", | ||
"symbol": "maxBTC", | ||
"logo_URIs": { | ||
"svg": "https://raw.githubusercontent.com/cosmos/chain-registry/master/neutron/images/maxBTC.svg" |
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 overall change to the rounded logo is good, but the old image (maxBTC (with the uppercase BTC)) will need to be removed, because we cannot have unused images.
"socials": { | ||
"website": "https://structured.money/", | ||
"twitter": "https://x.com/StructuredFi" | ||
"svg": "https://raw.githubusercontent.com/cosmos/chain-registry/master/neutron/images/maxbtc.svg" |
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.
cannot have svg under 'socials'
"svg": "https://raw.githubusercontent.com/cosmos/chain-registry/master/neutron/images/maxbtc.svg" | ||
}, | ||
"coingecko_id": "maxbtc", | ||
"images": [ |
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.
Have the images
property twice?
The schema is broken is its current state. Looks like there are duplicate object keys (socials, images) and a missing comma (really, it's just that svg is used where it shouldn't be) |
Updated logos for NTRN and Neutron.