Skip to content
This repository has been archived by the owner on Jan 20, 2024. It is now read-only.

dx: add global state management #170

Closed
wants to merge 1 commit into from
Closed

Conversation

iShibi
Copy link
Contributor

@iShibi iShibi commented Sep 11, 2022

@iShibi iShibi added the dx New developer experience related feature or request label Sep 11, 2022
@vercel
Copy link

vercel bot commented Sep 11, 2022

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
Copy link
Collaborator

ykdojo commented Sep 11, 2022

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
Copy link
Contributor Author

iShibi commented Sep 12, 2022

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
Copy link
Collaborator

ykdojo commented Sep 12, 2022

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
Copy link
Contributor

CerealPlayer commented Sep 16, 2022

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

@ykdojo
Copy link
Collaborator

ykdojo commented Sep 17, 2022

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

@iShibi iShibi closed this Sep 17, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
dx New developer experience related feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[DX] Add global state management
3 participants