SGiaccobasso commented 3 months ago

Description

Fix long loading times when searching for an address.

⛑️ What was done:

Set retry: false for TRPC query to get ERC20 token info. The query was being executed three times and each of these took a long time to complete, with one call being enough to get the same result.

Make the RPC calls to all chains concurrent for a much faster response.

Type of change

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update
[ ] Ready to merge

🧪 How to test:

Search for tokens, especially tokens that do not exist, and the error should come much earlier than in production.

🗒️ Notes:

[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules

⚖️ New vs Prod:

https://github.com/user-attachments/assets/953e9b39-e4d9-4cb9-ad0a-7b93d9f16407

The production app can take up to 35-45x longer to complete the search than the new version.

vercel[bot] commented 3 months ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
axelar-maestro	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Aug 15, 2024 8:45pm
axelar-registry	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Aug 15, 2024 8:45pm
axelar-ui	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Aug 15, 2024 8:45pm