turbine icon indicating copy to clipboard operation
turbine copied to clipboard

Take the localStorage out as parameter?

Open dmitriz opened this issue 8 years ago • 0 comments

Just an idea, motivated by the discussion in https://github.com/uzujs/uzu/issues/6#issuecomment-304519136, maybe taking out the local Storage here as external parameter would make the data flow more explicit: https://github.com/funkia/turbine/blob/master/examples/todo/src/TodoApp.ts#L69

Somehow I feel it does not really belong to the main app but comes from outside.

It might be interesting to compare with the implementation here: https://github.com/briancavalier/most-todomvc/blob/master/src/index.js#L75

dmitriz avatar May 29 '17 02:05 dmitriz