Skip to content

feat: do not eagerly fetch prices for tokens with no balance#3874

Merged
kev1n-peters merged 1 commit intodevelopmentfrom
dont-fetch-price-for-zero-balance-token
Oct 15, 2025
Merged

feat: do not eagerly fetch prices for tokens with no balance#3874
kev1n-peters merged 1 commit intodevelopmentfrom
dont-fetch-price-for-zero-balance-token

Conversation

@kev1n-peters
Copy link
Copy Markdown
Contributor

this was causing us to fetch balances for every pre-configured token immediately on page load even when some had no balance

this was causing us to fetch balances for every pre-configured token
immediately on page load even when some had no balance
@netlify
Copy link
Copy Markdown

netlify bot commented Oct 15, 2025

Deploy Preview for wormhole-connect ready!

Name Link
🔨 Latest commit 4fa06a4
🔍 Latest deploy log https://app.netlify.com/projects/wormhole-connect/deploys/68efe7218e9f3500080429d1
😎 Deploy Preview https://deploy-preview-3874--wormhole-connect.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@netlify
Copy link
Copy Markdown

netlify bot commented Oct 15, 2025

Deploy Preview for wormhole-connect-mainnet ready!

Name Link
🔨 Latest commit 4fa06a4
🔍 Latest deploy log https://app.netlify.com/projects/wormhole-connect-mainnet/deploys/68efe7211138930008b4554c
😎 Deploy Preview https://deploy-preview-3874--wormhole-connect-mainnet.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@kev1n-peters kev1n-peters merged commit 0e36024 into development Oct 15, 2025
20 checks passed
@kev1n-peters kev1n-peters deleted the dont-fetch-price-for-zero-balance-token branch October 15, 2025 18:29
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