Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Proposal data caching #453

Merged
merged 25 commits into from
Aug 13, 2021
Merged

Proposal data caching #453

merged 25 commits into from
Aug 13, 2021

Conversation

jdville03
Copy link
Member

@jdville03 jdville03 commented Aug 12, 2021

Implements caching strategy for proposal data using React Query as part of #423. (The issue description covers more than what is addressed by this PR so it should not be closed with just these updates.)

🥳 Adds

Updates

  • update tests to handle proposal data caching updates with react-query

@jdville03 jdville03 marked this pull request as ready for review August 13, 2021 20:30
@jdville03 jdville03 merged commit f01cd96 into main Aug 13, 2021
@jdville03 jdville03 deleted the proposal-data-caching branch August 13, 2021 20:30
github-actions bot pushed a commit that referenced this pull request Aug 13, 2021
* WIP cache proposal data

* WIP use queryClient

* WIP refactor with useQuery hooks

* apply caching strategy to governance proposals

* apply proposal data caching to individual proposal details

* Revert "apply proposal data caching to individual proposal details"

This reverts commit 3261578.

* Revert "Revert "apply proposal data caching to individual proposal details""

This reverts commit faa91be.

* WIP refetch for proposal actions updates

* fix proposal action transitions

* format comment

* fix proposal action transitions

* format comments

* WIP update tests

* clean up hook after caching update

* update useProposals unit tests

* update useGovernanceProposals unit tests

* update useDaoProposals unit test

* update comment

* WIP fix last of failing tests

* remove react query dev tools

* update queryClient import in setupTests

* update testing clear cache

* fixes outdated env vars being set

* fixes useProposalsVotes tests and makes slight improvements to code

* improve use of react queries

Co-authored-by: Jeremiah Naylor-Trein <jeremiahtrein@gmail.com> f01cd96
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants