What changed (plus any additional context for devs)
Limits Sentry tracking to crash reporting, startup time tracking, and explicit error reporting
Not fully clear why but Sentry is badly hurting app performance — limiting it to tracking crashes and startup times solves the problem
Fixes issues causing NFT data to be excessively recreated — some more improvements to be made but the big problems are resolved
Had to bump the nftsQueryKeypersisterVersion because the structure of data (from the useQuery hook) is now different with the NFT data transformation moved into the useQuery.
The structure of the data returned by the useLegacyNFTs wrapper hook is unchanged and fully backwards compatible — ideally we later remove the wrapper hook and migrate to the new NFT types.
Removes non-critical ENS-related hooks that were making a significant number of network requests
Removes unused code in App.js and in the WalletScreen component
Also:
Isolates the dapps data from UI components in the browser, because the dapps object is quite heavy and it's unnecessary for it to cause re-renders in search
Fixes APP-1555 Fixes APP-1566
What changed (plus any additional context for devs)
nftsQueryKey
persisterVersion
because the structure ofdata
(from theuseQuery
hook) is now different with the NFT data transformation moved into theuseQuery
.useLegacyNFTs
wrapper hook is unchanged and fully backwards compatible — ideally we later remove the wrapper hook and migrate to the new NFT types.App.js
and in theWalletScreen
componentAlso:
dapps
data from UI components in the browser, because thedapps
object is quite heavy and it's unnecessary for it to cause re-renders in searchScreen recordings / screenshots
What to test