add: OG ALF token

[sven-hash]

No description provided.

[sven-hash]

@Lbqds could it be possible to set a different name than the one used when deploying the token ?

[h0ngcha0]

@sven-hash Q: seems that ALF.svg is not used?

[h0ngcha0]

@Lbqds could it be possible to set a different name than the one used when deploying the token ?

The name has to match the name on-chain, there is a test in this repo to verify this:
https://github.com/alephium/token-list/blob/master/test/token-list/token-list.test.ts#L73-L79

[sven-hash]

@h0ngcha0 yes but it's not possible to have 2 tokens with the same name, let me know what I can do
And yes I can remove ALF.svg

[h0ngcha0]

@h0ngcha0 yes but it's not possible to have 2 tokens with the same name, let me know what I can do And yes I can remove ALF.svg

Interesting edge case, the newly added token has the same name as an existing one 🤔
@tdroxler WDTY? Does it make sense to e.g. introduce an optional onchain_name which falls back to name and use that to verify against the name in the token contract?

[sven-hash]

I removed the svg file

[tdroxler]

@h0ngcha0 mmm actually as we can have on chain tokens with same name/symbol etc, I guess we can accept them here, we should just modify this test to only check tokenId duplicates

[tdroxler]

hey @sven-hash you can have a look to #46 , I updated the test to allow tokens with same name/symbol, as it's the case on-chain.

I also fix our test since your PR wasn't failing while it should, since your data mismatch the on-chain one.

also I think we now need to change the logo name to the id of the token, it's less readable, but if we have tokens with same symbols, then we won't be able to have different images.

[sven-hash]

@tdroxler seems running great