IdiomaticReact
IdiomaticReact copied to clipboard
netgusto
Idiomatic React - Flux - REST app
You are using the `static` keyword for a non-method property here https://github.com/netgusto/IdiomaticReact/blob/e70187cb5aab570057a695cadbad6fdda05ed289/src/scripts/components/Todo/TodoList.js#L7. This not valid ES6 and fails babel's linter (at least in my setup). Per [this version of the...
Thank you for open-sourcing this repo! I have a quick observation regarding the use of Immutable.js in the example. ``` todos = Immutable.OrderedMap() getTodos() { return this.todos.toArray(); } ``` As...
Instead of fetching todo's on componentDidMount you could declare them on the class and trigger them inside the router. The reason for doing this is if you call actions inside...
Lots of nice bits in here. The folder structure is a little hard to navigate. I'm quite a big fan of this document about folder structure: https://gist.github.com/ryanflorence/daafb1e3cb8ad740b346 Although its overkill...