search

csdojo-defaang / defaang

A website that will curate recently-asked interview questions from FAANG+. Currently inactive. Check out: https://github.com/ykdojo/OpenStream
MIT License
509 stars 120 forks source link

dx: add global state management #170

Closed iShibi closed 2 years ago

iShibi commented 2 years ago

Closes #169

vercel[bot] commented 2 years ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
defaang ✅ Ready (Inspect) Visit Preview Sep 11, 2022 at 8:30AM (UTC)
ykdojo commented 2 years ago

I was doing some research on this, and I'm wondering if we need Redux for this app.

What do you think about simply using React Context for it, instead?

iShibi commented 2 years ago

What do you think about simply using React Context for it, instead?

Redux Toolkit provides a better and easier developer experience. Updating the shared state is also easy and then there is the Redux devtool for debugging purposes. The website is going to be large and dynamic in nature. Redux Toolkit is best suited for our use case.

ykdojo commented 2 years ago

I think Redux is just way too complex for what we have right now. All we need is basically a way to keep track of the user session and the user.

More context.

We've had a discussion about it on #share-resources on Discord, too.

CerealPlayer commented 2 years ago

I agree with YK. Redux is kind of an old giant. For this app's purpose React Context could definitely work

ykdojo commented 2 years ago

@iShibi are you still working on this? If not, I'll close it and go back to #167.