Start adding Redux for data management.

Review Request #9678 — Created Feb. 19, 2018 and submitted

david
student-sonar
master
0c0a46a...
student-sonar

Backbone models and collections are not a great way to store data for a
react application. This change starts the (probably lengthy) process of
migrating towards a Redux data management model. The first thing which
is really migrated here is the project ideas, which is one of the most
isolated pieces of data.

In order to match things up here, this also includes the start of
migration for the users and the "manage" state. Those are only very
partially migrated, so there's a lot more work to do there.

Navigated around through the three views that use projects (the project
list, the single project view, and the user detail view) and saw that
everything worked as expected.

Description From Last Updated

Now that we're using redux, you may want to include userType etc in the state and hydrate from the JS ...

brenniebrennie

Col: 2 Unnecessary semicolon.

reviewbotreviewbot

Col: 52 Regular parameters should not come after default parameters.

reviewbotreviewbot

Col: 59 Regular parameters should not come after default parameters.

reviewbotreviewbot

Col: 59 Regular parameters should not come after default parameters.

reviewbotreviewbot

Col: 52 Regular parameters should not come after default parameters.

reviewbotreviewbot

Col: 59 Regular parameters should not come after default parameters.

reviewbotreviewbot

Col: 59 Regular parameters should not come after default parameters.

reviewbotreviewbot
Checks run (1 failed, 1 succeeded)
flake8 passed.
JSHint failed.

JSHint

brennie
  1. 
      
  2. lib/frontend/project.jsx (Diff revision 1)
     
     

    Now that we're using redux, you may want to include userType etc in the state and hydrate from the JS that gets splatted into the page.

    1. Yeah, that's in the pipeline.

  3. 
      
brennie
  1. Ship It!

  2. 
      
david
Review request changed

Commit:

-04e38af052e4f49192a8e3612fe49092d6f5707f
+0c0a46a81c3e3e75c77d26c3af878376b92d43ab

Diff:

Revision 2 (+447 -232)

Show changes

Checks run (1 failed, 1 succeeded)

flake8 passed.
JSHint failed.

JSHint

brennie
  1. Ship It!
  2. 
      
david
Review request changed

Status: Closed (submitted)

Change Summary:

Pushed to master (e057d1d)
Loading...