chainapsis / keplr-chain-registry

Protocol for Integrating Non-Native Chains to Keplr
https://chains.keplr.app
Other
335 stars 502 forks source link

Add tucana birdee-1 testnet #718

Closed garymeso closed 3 months ago

vercel[bot] commented 3 months ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
keplr-chain-registry ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 20, 2024 1:46am
garymeso commented 3 months ago

HI @editaahn , this PR is ready to review, could you review it? Thank you.

garymeso commented 3 months ago

Hi @editaahn , why the validation job shows: Invalid image url: https://raw.githubusercontent.com/chainapsis/keplr-chain-registry/main/images/tucana/chain.png?

It will be available after merge. You may check through the github content url of my forked repo: https://raw.githubusercontent.com/garymeso/keplr-chain-registry/add-tucana-birdee-testnet/images/tucana/chain.png

editaahn commented 3 months ago

Hi! Sorry for the late response. You should update the image directory following the json file name. Also, please make sure the image size is 256 * 256px.

garymeso commented 3 months ago

OK, thank you for the reminding. Image updated, please re-trigger the validation job, thank you. @editaahn

editaahn commented 3 months ago

Seems like you didn't change the name of the image directory to birdee

garymeso commented 3 months ago

Is it a new rule? I noticed there are some testnets chains use the same image as their mainnets, e.g. https://github.com/chainapsis/keplr-chain-registry/blob/main/cosmos/theta-testnet.json.

Fixed. Please review again thanks.