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.

Sign up for GitHub

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

Jump to bottom

Add chain dex #19

Merged
merged 30 commits into from
Feb 8, 2022
Merged

Add chain dex #19

merged 30 commits into from
Feb 8, 2022

Conversation

on-myown
Copy link
Contributor

No description provided.

Jason Liu and others added 19 commits January 12, 2022 01:14
add few more chains and expanse chain png
5da74ee
add few more chains and expanse chain png
de0ca01
add cronaswap on cronos chain
70690ce
add bridge comment for cronos chain
19bd4b2
add DEXs on fuse chain
688df9d
add ubeswap on celo chain
8348a72
add sushiswap on celo chain
84c6dce
add icons for some chains
7967346
add moonbeam chain icon
88a5e61
add default tokens for moonbeam
27ed810
add chains icons and information
416bed6
@maxklenk
fix: sort multicall addresses
c901a17
add standard protocol for metis network
6ad21eb
Merge branch 'main' of https://github.com/lifinance/types into add-mo…
d782512 
…re-chain
Merge branch 'add-more-chain' of https://github.com/lifinance/types i…
2718896 
…nto add-more-chain
feat: add some information about standard protocol for metis network
a5cd0b2
fix: fix fuse network information
046ed09
fix: add logoURIs to supported chains
b980260
fix: merge 3 commits to one
8210896
mathiasmoeller
mathiasmoeller requested changes Jan 24, 2022
View reviewed changes
Copy link
Member

@mathiasmoeller mathiasmoeller left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So many new tokens 💪

Regarding the images (I can't comment them directly unfortunately):
Some images like expanse.png should be cropped. They have a large border and they will appear pretty small when shown in the website.

src/multicall.ts Show resolved Hide resolved
@on-myown
Copy link
Contributor Author

So many new tokens 💪

Regarding the images (I can't comment them directly unfortunately): Some images like expanse.png should be cropped. They have a large border and they will appear pretty small when shown in the website.

@mathiasmoeller , I am gonna crop it to 256 x 256. will it be fine for website ?

@mathiasmoeller
Copy link
Member

mathiasmoeller commented Feb 1, 2022 •
edited

@mathiasmoeller , I am gonna crop it to 256 x 256. will it be fine for website ?

Yes I think so, we only display them as very small icons right now :)

@maxklenk maxklenk merged commit a898b58 into main Feb 8, 2022
@maxklenk maxklenk deleted the add-chain-dex branch February 8, 2022 11:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mathiasmoeller mathiasmoeller mathiasmoeller requested changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@on-myown @mathiasmoeller @maxklenk