graph-explorer icon indicating copy to clipboard operation
graph-explorer copied to clipboard

[Project] Database queries should use React Query

Open kmcginnes opened this issue 10 months ago • 0 comments

Database queries go through a custom fetch process within React code. This requires a lot of boilerplate context, hooks, callbacks, memos, etc of React specific code. But this logic amounts to a simple async fetch call with a set of parameters.

Benefits

  • React Query has built in caching that does not depend on HTTP caching
  • React Query has configurable retries
  • Moving connector & explorer logic to functions will make them easier to understand, test, and might improve React rendering performance
  • Moving connector & explorer logic to functions will make it easier to move this logic to the server or create an independent package/library

Related Issues

  • Prerequisite #357
  • Addresses #269
  • Addresses #256

Preferred Solution

  • [x] #335
  • [x] #340
  • [x] #338
  • [x] #339
  • [ ] #336
  • [x] #337
  • [ ] #443
  • [ ] #479

[!TIP] Please use a 👍 reaction to provide a +1/vote.

This helps the community and maintainers prioritize this request.

[!IMPORTANT] If you are interested in working on this issue or have submitted a pull request, please leave a comment.

kmcginnes avatar Apr 15 '24 19:04 kmcginnes