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

(FE) Refactor react-query and redux #104

Merged
merged 5 commits into from
Jan 25, 2024
Merged

Conversation

devleejb
Copy link
Member

What this PR does / why we need it:

Avoid using the built-in caching of react-query and instead store the data in Redux. Multiple invocations of the same react-query hook in different parts of the application result in redundant calls and code duplication.

Which issue(s) this PR fixes:

Fixes #91

Special notes for your reviewer:

Does this PR introduce a user-facing change?:

NONE

Additional documentation:


Checklist:

  • Added relevant tests or not required
  • Didn't break anything

@devleejb devleejb added the cleanup 🧹 Paying off technical debt label Jan 25, 2024
@devleejb devleejb self-assigned this Jan 25, 2024
@devleejb devleejb merged commit b294943 into main Jan 25, 2024
1 check passed
@devleejb devleejb deleted the 91-fe-refactor-react-query--redux branch January 25, 2024 07:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cleanup 🧹 Paying off technical debt
Projects
None yet
Development

Successfully merging this pull request may close these issues.

(FE) Refactor react-query & redux
1 participant