-
Notifications
You must be signed in to change notification settings - Fork 2
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
change live-peers to a react redux app? #107
Comments
👍 I've thought about this quite a few times! If I can find time, I'd be happy to review PRs/answer questions about the current API and key features for whoever would like to develop this in react/redux. I propose;
To start the server, and serve the react-bundled code from a designated directory, and...
To start the server, and serve the current static js assets we have in this repo (for vanilla implementation of the app). This would allow devs to contribute to this repo in both areas; vanilla implementation & react/redux (and eventually Elm :p). |
If you guys do this, I suggest it would be great to document your reasoning too 👍🏻 |
Now I'm learning Elm again & wanna use that 😅 Unless we ever actually put Elm into the curriculum permanently, React/Redux would make it easier for students to jump into the codebase and encourage contributions. 🤔 So part of this is obviously using a project I like and care about to help me learn whatever new piece of Tech I'm playing with at the time...but if we do actually wanna use this in FAC at some point, we might as well choose a framework that means we're not gonna experience runtime errors... 😜 I'm thinking of trying to Elm-ify this at the end of this week. If I do, I'm thinking I would make a new repo & steal the assets from this one. But would that be ok with you @njsfield ? |
@njsfield I know we talked about this before. How would you feel about using react and redux for live-peers?
@esraajb @njsfield Would you mind if this repo changed? Should we keep this one with Hapi & make a new one?
The text was updated successfully, but these errors were encountered: