sfbrigade / datasci-sba

Solving problems with the Small Business Administration
10 stars 18 forks source link

Adding redux for clientside state management #47

Closed avdonovan closed 7 years ago

avdonovan commented 7 years ago

1. Brief Summary of what this PR accomplishes (140 characters or less. If you find trouble describing what you are doing in this length, consider breaking the PR into multiple ones.)

Adds the Redux JS library to manage the clientside state in the prototype visualization of regions.

2. Link to Trello Ticket

https://trello.com/c/PgLlDuMx/43-refactor-region-visualization-to-use-redux-or-something-similar

3. More detailed description and other questions to address in code review

Redux adds a standard pattern for managing clientside state -- e.g. which variables are selected, what the filter state is -- that makes it easier to scale as the app gets more complicated. It's also a pretty common standard that would make it easier for new team members to understand the structure and contribute sooner.

4. Remember to tag reviewers!