Skip to content

Remove Redux completely

Axel Bocciarelli requested to merge remove-redux into main

I'm converting the remaining Redux code to zustand, as discussed. Even though I think the state needs a bit of refactoring, I left it exactly the same for now (except for some simple renaming).

For reference, here are the issues that are worth refactoring later:

  • The directoryPath state is duplicated inside the imagePath state.
  • When we load an image path from the URL, we call two store setters ‑- we could call a dedicated setter instead to avoid consecutive re-renders.
  • Ideally, I'd like the imagePath state to be moved to the URL, but I need to think about it a bit more.

Merge request reports