Skip to content

Add getTokens()#86

Merged
chapati23 merged 6 commits intomainfrom
feat/add-get-tokens
Oct 2, 2025
Merged

Add getTokens()#86
chapati23 merged 6 commits intomainfrom
feat/add-get-tokens

Conversation

@chapati23
Copy link
Copy Markdown
Contributor

Description

We're hardcoding token addresses in the frontend right now. This adds a getTokens() function that will iterate over our tradablePairs.ts cache and extract a unique list of tokens.

It also adds a yarn tokens script that will print out a list of tokens across all networks (default) or by chain when called with yarn tokens -n celo-sepolia

Copy link
Copy Markdown
Contributor

@Andrew718PLTS Andrew718PLTS left a comment

Choose a reason for hiding this comment

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

Code Review: LGTM, left a couple of minor comments.
Testing: It works fine, but it's failed to get tokens for base:
image

@Andrew718PLTS Andrew718PLTS self-requested a review October 2, 2025 09:44
@chapati23 chapati23 merged commit bd9558e into main Oct 2, 2025
1 check passed
@chapati23 chapati23 deleted the feat/add-get-tokens branch October 2, 2025 09:50
@chapati23 chapati23 mentioned this pull request Oct 2, 2025
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.

2 participants