Closed ivan-aksamentov closed 2 years ago
This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.
🔍 Inspect: https://vercel.com/hodcroftlab/covariants/F9b4BDw1TNtgpc14hYmXRr1RtNPt
✅ Preview: https://covariants-git-refactor-change-redux-to-recoil-hodcroftlab.vercel.app
This swaps state management framework from
redux
torecoil
.Recoil is more modern, ergonomic, plays nicely with React hooks, and, hopefully with async rendering.
I am planing to introduce some serious modifications to the app state shortly, so it is better to make the swap before that, to avoid basically writing the thing twice.
So far we used redux for just a couple of things:
Both are kind of redundant also. I made navbar to use the pathname from the Next.js router, and moved dropdown state into a recoil atom.
In this PR, there should be no visual or behavioral changes in the application. If there is, it's a bug.