Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Look into Redux #10

Open
K-Ho opened this issue Oct 16, 2018 · 2 comments
Open

Look into Redux #10

K-Ho opened this issue Oct 16, 2018 · 2 comments

Comments

@K-Ho
Copy link
Member

K-Ho commented Oct 16, 2018

Redux could be a better way to manage state, since we pass so much information in as props and have a bunch of passed in functions that we call (e.g. functions passed in from Network.Js to be called by the buttons in controls.js)

@K-Ho K-Ho created this issue from a note in Visualization 2.0 (To do) Oct 16, 2018
@K-Ho
Copy link
Member Author

K-Ho commented Oct 20, 2018

Our App's State will basically only need: {
time: 10.001,
messages: [{each message contains: transaction contents, from, to, sendTime, rcvTime}]
}

When clicking on something like "double spend", instead of having to call a function that has been based down through props, can just call a Redux Action that will add the 2 messages to the state.

@K-Ho K-Ho removed this from To do in Visualization 2.0 Oct 24, 2018
@K-Ho
Copy link
Member Author

K-Ho commented Nov 21, 2018

Helpful overview of other libraries to use: https://www.robinwieruch.de/essential-react-libraries-framework/

@K-Ho K-Ho added this to In progress in Visualization 2.0 Nov 27, 2018
@K-Ho K-Ho moved this from In progress to To do in Visualization 2.0 Mar 8, 2019
@K-Ho K-Ho removed this from To do in Visualization 2.0 Jul 11, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant