Skip to content
This repository has been archived by the owner on Apr 17, 2023. It is now read-only.

Aggregate Hooks Issue #968

Open
kingsleyzissou opened this issue Nov 30, 2020 · 1 comment
Open

Aggregate Hooks Issue #968

kingsleyzissou opened this issue Nov 30, 2020 · 1 comment
Milestone

Comments

@kingsleyzissou
Copy link
Contributor

Bug Report

There are quite a few issues with the hooks. We have state in too many places and possibly the best solution is to strip back
the hooks, use the remote to fill the store:

  • if we keep db as state provider
  • remote will be refilling things
  • and UI will just read state when it needs ( like react setState)

List of issues:

@kingsleyzissou kingsleyzissou added this to the Demo MVP milestone Nov 30, 2020
@wtrocki
Copy link
Contributor

wtrocki commented Nov 30, 2020

Yes. While this is 100% way to go we might actually really easily drop current implementation and get something from scratch. This is due to the fact that there might be some strange approaches in current hooks that might still leak out (useReducer etc.) so.. I would be easier to start from scratch.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants