-
Notifications
You must be signed in to change notification settings - Fork 12
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
Tracking Reactification #126
Comments
I like it! I'm going to be looking through the React courses @jessethach sent me, so this would be a fun thing to tackle after the hackathon. Were you wanting to make this a solo venture or looking for help on tasks? |
I'm game for any help people want to lend! I'm doing it in a reactify branch on the main repo so that anyone on the team can take a look or contribute 😄 |
I'm in! But I do need some time to catch up on the react magic after the hackathon. |
Me too! I've been meaning to add a games section too! 😃
|
I have some ideas. I can send some PRs for review soon. |
@ltegman we can probably turn the |
@jessethach Would that be so we could store the button state in redux and use that to highlight the currently pressed button? |
@ltegman Definitely – same could be done with the motion-socket file. |
I've fixed the tests by removing the no longer used (and therefore broken) router tests. Some tests fail when run in gulp because it runs so much slower, so that may be something to look into, but it's not high on my priority list. |
Did someone say React? |
I'm most of the way done with porting the basic skeletons of the pages. Still will need to wire up the actions and handle stuff like graphs and the color picker in react Thinking about switching to this for color picking: http://casesandberg.github.io/react-color/ Thinking about using this for wiring up all our UI actions that use sockets: https://www.npmjs.com/package/redux-socket.io |
Primary motivation for switching this to React is that turning the client-side into a SPA is an easy way to stop us from spamming sockets and event listeners on our server-side, which we currently do every time someone navigates to a new page. It will also allow us to keep accelerometer history when the user navigates away from that page.
Here I'm tracking work that needs to be done for this milestone:
Components
Misc
Stretch
The text was updated successfully, but these errors were encountered: