[bug] Advanced tools USD notional is incorrect for some tokens #1025

danielisaacgeslin · 2024-08-16T10:13:47Z

Makes different api calls to coingecko depending on the token being native or not
Hardcoded coingecko id maps is no longer needed (support for new tokens)

native token

non native token

src/utils/telemetry.ts

[bug] Advanced tools USD notional is incorrect for some tokens

a252855

danielisaacgeslin temporarily deployed to Cloudflare-Testnet August 16, 2024 10:13 — with GitHub Actions Inactive

danielisaacgeslin temporarily deployed to Cloudflare-Preview August 16, 2024 10:13 — with GitHub Actions Inactive

danielisaacgeslin temporarily deployed to Cloudflare-Preview August 16, 2024 10:21 — with GitHub Actions Inactive

danielisaacgeslin temporarily deployed to Cloudflare-Testnet August 16, 2024 10:22 — with GitHub Actions Inactive

fix on token address

9d4f9d3

danielisaacgeslin temporarily deployed to Cloudflare-Testnet August 16, 2024 10:39 — with GitHub Actions Inactive

danielisaacgeslin temporarily deployed to Cloudflare-Preview August 16, 2024 10:39 — with GitHub Actions Inactive

danielisaacgeslin temporarily deployed to Cloudflare-Testnet August 16, 2024 10:47 — with GitHub Actions Inactive

danielisaacgeslin temporarily deployed to Cloudflare-Preview August 16, 2024 10:48 — with GitHub Actions Inactive

danielisaacgeslin requested review from sebastianscatularo and yuli-ferna August 16, 2024 11:03

sebastianscatularo approved these changes Aug 20, 2024

View reviewed changes

src/utils/telemetry.ts Show resolved Hide resolved

danielisaacgeslin merged commit 090cd7a into advanced-tools Aug 20, 2024
8 checks passed

sebastianscatularo deleted the bug/USD-notional-some-tokens branch August 30, 2024 14:32

Provide feedback