tableflip / tune

:trumpet: Client for the TABLEFLIP static site generator
0 stars 1 forks source link

Adds Collections #24

Closed richsilv closed 8 years ago

richsilv commented 8 years ago

Requires #20 and #22

alanshaw commented 8 years ago

👓

alanshaw commented 8 years ago

I've not had time to run this but it looks good. My main concern is that we're occasionally using the store via import and not via the redux connect function. I think we should be sticking to the "redux pattern" wherever possible. If nothing else it'll make our components much more testable.

Please rebase and merge :thumbsup:

richsilv commented 8 years ago

All good points! This was the first go at using Redux and so I haven't stuck to the connect pattern as I should.

I'll update where relevant, rebase and merge.