Closed shane-moore closed 9 months ago
Thank you for providing the follow-up on the Automation script to make the PR process easier. Our development team has read the Issue opened on the chainlist and we are a bit skeptical about using the Automcaiton script because by looking at the limitation, script itself does not ensure to include all of the data that requires for us to operate products and services.
Therefore, even with the automated processes, human verification is still required, making it necessary for us to regularly undertake this take. So, we would still like to ask the team to submit the PR on the assets that the team would like to add. For the urgent ones, like last time, please inform us through the telegram so that we can work on it as quickly as possible.
Scope
chain/injective/assets.json
file. The automation script will live within injective's open-sourceinjective-ts
repo.Sandbox environment
injective-ts
and added the script, which can be found herecosmostation/chainlist
and ran the automation script frominjective-ts
, which opened a test PR to the forkedcosmostation/chainlist
repoLimitations of the script
chain/injective/assets.json
except for adding acoingeckoId
if not present. Otherwise, the script ONLY appends new assets tochain/injective/assets.json
origin_chain
will always have a value since we don't always have this on our end. Please check this when reviewing each PR tocosmostation
, if it's a required fieldorigin_type
as we do not store that info ininjective-ts
hardcoded assets listdecimals
andcoingeckoId
are included on the new assets added in the PR tocosmostation
, as we need those for defillama calc. else, please add if possibleibc
orbridge
token type properties of an asset for now as I'm trying to keep it simple. If needed, please add manually for nowTodo
injective-ts
repo and use it to create a PR to the canonicalcosmostation/chainlist
repo