Stop using loader functions, part 1

jessepinho commented 1 week ago

This PR goes through a bunch of routes where we're using react-router loaders, and replaces loader functionality with ZQuery. Note that this means some assumptions have changed about when data is available. react-router loaders wait to render their route components until data has loaded, which means your component can assume the data is already present. Since we're now effectively loading data from useEffect() (by way of ZQuery), we can't assume that data is present just because a component is rendered. So you'll see some instances of code that accounts for undefined where we didn't previously have to.

One of the benefits of this change is that route changes are instant. There's no lag time between when you click on a link to a route, and when the route changes, like there was before when route changes were blocked by loaders.

Before (note the delayed route changes after I click)

https://github.com/penumbra-zone/web/assets/1121544/7fbb50a0-4acf-4789-b0d2-51d99c49fcef

After (note the instant route changes after I click)

https://github.com/penumbra-zone/web/assets/1121544/22a7dba1-3ae8-4fef-a753-5e3f8479669e

In this PR

Remove the balances loader and create a balances slice with a balancesByAccount ZQuery object.
Remove the Send form's loader and create a transferableBalancesResponses ZQuery object in the send state.
Create a shared Zustand slice and added a stakingTokenMetadata ZQuery object to it, to be used by the send form and the IBC UIs.
Remove the staking page's loader; in its place, use state.shared.stakingTokenMetadata, and combine the accountSwitcherFilter and unstakedTokensByAccount into a single ZQuery object, since they both depend on the same data.
Remove the transactions' page's loader; in its place, convert its slice to use a ZQuery object.

Future PRs

Convert the SwapLoader and IbcLoader to ZQuery. Those look a bit involved and this PR is already touching a lot of files, so I haven't done those yet.
Convert more of the staking slice to ZQuery. That would be a bit more involved as well, and doesn't involve getting rid of loaders, so I figured I'd punt on that for now.

Relates to #1243