jaredly / jaredly.github.io

node.js python data science golang philosophy faith. in no particular order
jaredly.github.io
10 stars 13 forks source link

Expand /* ... */ in the upper part of TodoApp_1_3.re #9

Closed fikriauliya closed 7 years ago

fikriauliya commented 7 years ago

First of all, thx for the tutorial!

screen shot 2017-07-09 at 9 52 32 pm

I was a little bit confused on step 1_3 because of the unbounded update error. Based on the screen shot above, the only expected diffs from step 1_2 are those 10 lines. But in fact, there is a change on render fun parameter as well:

from fun {state: {items}} to fun {state: {items}, update}

It would be clearer if the snippet for 1_3 is to be expanded up, at least up to the render method signature.

pjhenning commented 7 years ago

Ran into this too, I was convinced it was a bug until I took a look at the reason-react todomvc example and realized update needs to be specified as a parameter haha.

And yes thanks so much for putting this together, this tutorial and the others you've put up have been helping me a lot!

bado22 commented 7 years ago

I ran into the same thing but was trying to figure out how to fix it. Thanks for posting the solution here @fikriauliya & @gandulio!

jaredly commented 7 years ago

fixed, finally 😅