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

Use SWR #131

Draft
wants to merge 3 commits into
base: master
Choose a base branch
from
Draft

Use SWR #131

wants to merge 3 commits into from

Conversation

shaunanoordin
Copy link
Member

PR Overview

This PR changes the data-fetching action to use SWR. This follows the recommendation of Jim, to avoid using setState()s in useEffect()s.

Status

⚠️ EXPERIMENTAL

@shaunanoordin shaunanoordin marked this pull request as draft July 20, 2023 17:36
@eatyourgreens
Copy link
Contributor

This article about data-fetching starts with a good explanation of the bugs that exist in your useEffect hooks.

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