graph-explorer
graph-explorer copied to clipboard
[Project] Database queries should use React Query
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.