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

Update the Logo of MRSMIGGLES #1345

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

MrsMigglesOnBase
Copy link

We have been listed on Coingecko and through that, our token was automatically imported into Metamask. The logo has changed in the meantime, hence this PR to update the token logo to the current Version. The committed logo is also visible on Coingecko.

Website: https://mrsmigglesbase.com/

Coingecko: https://www.coingecko.com/en/coins/mrs-miggles

Twitter: https://x.com/MrsMiggles_Base

@MrsMigglesOnBase MrsMigglesOnBase requested review from MRabenda and a team as code owners August 18, 2024 19:01
@MRabenda MRabenda added the Base label Aug 19, 2024
@MRabenda
Copy link
Collaborator

SVG image is required

@MrsMigglesOnBase
Copy link
Author

SVG image is required

the svg has been uploaded and the contract-map.json changed to the correct file extension. The png has been removed

@MrsMigglesOnBase
Copy link
Author

Hello again,

people have been bugging me multiple times about the wrong (old) logo in MetaMask. Could you review this again and merge the pull request please?

Kind regards

@MRabenda
Copy link
Collaborator

Hey @MrsMigglesOnBase

This repo supports only Ethereum, to update token on Base would you guys be able to add the token to the following lists?:
Coingecko: https://tokens.coingecko.com/polygon-pos/all.json
sushiswap: https://token-list.sushi.com/
UniswapLabs: https://gateway.ipfs.io/ipns/tokens.uniswap.org

@MrsMigglesOnBase
Copy link
Author

MrsMigglesOnBase commented Nov 19, 2024 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants