Skip to content
This repository has been archived by the owner on Mar 23, 2023. It is now read-only.

Flux To Do Tutorial #37

Closed
sunnyrjuneja opened this issue Aug 22, 2014 · 2 comments
Closed

Flux To Do Tutorial #37

sunnyrjuneja opened this issue Aug 22, 2014 · 2 comments

Comments

@sunnyrjuneja
Copy link
Contributor

This is related to #31. The code in the Flux Todo MVC has a different dispatcher than the tutorial on the site.

This is pretty confusing because I had the tutorial open side by side with the code.

I would be happy to send a PR to update the site but I'm not sure if adding the new dispatcher is the right decision since the previous dispatcher seems shorter and simpler.

@fisherwebdev
Copy link
Contributor

I would highly recommend the new dispatcher. It's the one we use at Facebook. The other one was just a temporary stand-in while we were getting approval to release the real code. The new dispatcher has been battle-tested in many different applications at Facebook and we trust it. The other one was a totally naive implementation I created just to get the ball rolling, after Jing Chen had presented Flux at the F8 conference. Some people have enjoyed the async nature of the previous one, but I personally would not use it in production.

All that said, yes I need to update the tutorial and I will do that.

@fisherwebdev
Copy link
Contributor

Pull request sent to ReactJS: facebook/react#2080

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants