fix useContractReads state dumping

[jordanlesich]

Description

Adds keepPreviousData : true to useContractReads to useVotes hook and ProposalActions

Motivation & context

wagmi useContractReads hook was forcing page re-renders due to a bug with state/caching persistence

wevm/wagmi#2878

Took a bit of digging, but I was able to find a parameter that would stop this from happening

keepPreviousData : true

This solves both issues:

1. Proposal description and title being deleted on first login on network switch
2. Vote modal closing and deleting vote reason data on vote page.

Code review

Type of change

• 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)

Checklist

• I have done a self-review of my own code
• Any new and existing tests pass locally with my changes
• My changes generate no new warnings (lint warnings, console warnings, etc)

[vercel]

@jordanlesich is attempting to deploy a commit to the Nouns Builder Team on Vercel.

A member of the Team first needs to authorize it.

[vercel]

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

Name	Status	Preview	Updated (UTC)
nouns-builder	✅ Ready (Inspect)	Visit Preview	Oct 23, 2023 2:11am
testnet-nouns-builder	✅ Ready (Inspect)	Visit Preview	Oct 23, 2023 2:11am

[isaaczaak]

Screen.Recording.2023-10-23.at.2.10.58.pm.mov

just want to make sure that "Vote modal closing and deleting vote reason data on vote page." refers to the issue that i'm experiencing in the recording attached @jordanlesich

[isaaczaak]

Clarified that issue #2 that this PR is trying to solves for is related to only to when a user is voting on a proposal when connected to a different network.