Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

#816 | change the strategy to only fetch contracts from cache #849

Closed
wants to merge 9 commits into from

Conversation

Viterbo
Copy link
Collaborator

@Viterbo Viterbo commented Sep 6, 2024

Fixes #816

Description

The current implementation has a minimal difference. Now, each time we need to display an address, we take the possible contract metadata from the local cache, delegating to the indexer API the responsibility of populating that cache on each query.

This will have a real optimization impact on the token holders and home pages after the following two PRs are deployed on the indexer API side:

Test scenarios

@Viterbo Viterbo self-assigned this Sep 6, 2024
Copy link

netlify bot commented Sep 6, 2024

Deploy Preview for dev-mainnet-teloscan ready!

Name Link
🔨 Latest commit ba76a94
🔍 Latest deploy log https://app.netlify.com/sites/dev-mainnet-teloscan/deploys/66db0890690acb0008ed081b
😎 Deploy Preview https://deploy-preview-849--dev-mainnet-teloscan.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 site configuration.

Copy link

netlify bot commented Sep 6, 2024

Deploy Preview for testnet-teloscan ready!

Name Link
🔨 Latest commit ba76a94
🔍 Latest deploy log https://app.netlify.com/sites/testnet-teloscan/deploys/66db0890c4f0a400088e1513
😎 Deploy Preview https://deploy-preview-849--testnet-teloscan.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 site configuration.

Copy link

netlify bot commented Sep 6, 2024

Deploy Preview for teloscan-stage ready!

Name Link
🔨 Latest commit ba76a94
🔍 Latest deploy log https://app.netlify.com/sites/teloscan-stage/deploys/66db08901b08ab0007960a57
😎 Deploy Preview https://deploy-preview-849--teloscan-stage.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 site configuration.

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