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

add terraport token #35

Merged
merged 7 commits into from
May 11, 2024
Merged

add terraport token #35

merged 7 commits into from
May 11, 2024

Conversation

Man12C
Copy link
Contributor

@Man12C Man12C commented Apr 4, 2024

add terraport token on terra classic chain

@HeesungB
Copy link
Collaborator

HeesungB commented Apr 5, 2024

@Man12C Could you change image url path?
I think the path below is correct

https://raw.githubusercontent.com/chainapsis/keplr-contract-registry/main/images/columbus/terraport.png

@Man12C
Copy link
Contributor Author

Man12C commented Apr 5, 2024

@Man12C Could you change image url path? I think the path below is correct

https://raw.githubusercontent.com/chainapsis/keplr-contract-registry/main/images/columbus/terraport.png

Changed as requested, thank you!

@HeesungB
Copy link
Collaborator

HeesungB commented Apr 5, 2024

@Man12C Could you rename image, url as symbol name?

@Man12C
Copy link
Contributor Author

Man12C commented Apr 5, 2024

@Man12C Could you rename image, url as symbol name?

Done

@HeesungB
Copy link
Collaborator

HeesungB commented Apr 8, 2024

@Man12C Can I ask why did you remove json file?

@Man12C
Copy link
Contributor Author

Man12C commented Apr 11, 2024

Can you merge it?

@Man12C Can I ask why did you remove json file?

@HeesungB
Copy link
Collaborator

@Man12C Could you resize image?

Reduce image size to 512x512 or smaller (expected: 512x512, actual: 630x640)

@Man12C
Copy link
Contributor Author

Man12C commented Apr 19, 2024

@Man12C Could you resize image?

Reduce image size to 512x512 or smaller (expected: 512x512, actual: 630x640)

Done

@Man12C
Copy link
Contributor Author

Man12C commented May 11, 2024

Can you merge it?

@HeesungB HeesungB merged commit e53a0e0 into chainapsis:main May 11, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants