search

tasitlabs / tasit-apps

Native mobile Ethereum dapps for mainstream users
https://tasit.io/
MIT License
36 stars 9 forks source link

Decide between redux-thunk, redux-saga, or neither for async data fetching #20

Open pcowgill opened 5 years ago

marcelomorgado commented 5 years ago

@pcowgill commented:

I think React context API is worth considering in the medium term.

Refs: https://github.com/tasitlabs/tasit/pull/93#issuecomment-462464898

marcelomorgado commented 5 years ago

Seems that redux docs recommends keep actions as plain functions and use middleware (thunk/saga) to handle with fetching data. What do you think? Refs: https://github.com/tasitlabs/tasit/pull/93#discussion_r255698391

marcelomorgado commented 5 years ago

TODO Move functions that interact with blockchain to actions / middlewares.

pcowgill commented 5 years ago

Depending on what we decide in issue https://github.com/tasitlabs/tasit/issues/197 , this one might become obsolete