Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
I removed the
pollInterval
setting for all graphql queries so that we are doing any http requests to the graphql BE only on page load. This can be easily noticed on the/blocks
page, where we were previously polling a lot of data every 30 seconds.I also checked the
useBlockNumber
custom hook that we have in/staking/src/services/eth.tsx
. It is used on most of the pages and although it often causes re-renders in components that use it (because the block number changes every ~15 seconds), I didn't notice it causing any side effects like extra http requests happening. That's why I didn't make any changes inuseBlockNumber
.Testing Instructions
/
(home), or/blocks
, or any other page where we were previously polling the graphql queries