Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[docs] Update Flux TodoMVC Tutorial #2080

Merged
merged 3 commits into from
Aug 28, 2014
Merged

Conversation

fisherwebdev
Copy link
Contributor

I was reminded by @whatasunnyday in the Flux repo's issue #37 that the Flux TodoMVC tutorial needs to be updated to reflect the latest dispatcher we have released. The tutorial had previously used a dispatcher based on promises, and this was confusing for newcomers.


We'll use the dispatcher from the [Flux GitHub repository](https://github.com/facebook/flux), but let's go over how to get it into our project, how it works and how we'll use it.

The dispatcher's source code is written in [ECMAScript6](https://github.com/lukehoban/es6features), the future version of JavaScript. To use the future of JS in today's browser's we need to transpile it back to a version of JS that browsers can use. We perform this build step, transpiling from ES6 into common JavaScript, using npm.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: "ECMAScript 6", not "ECMAScript6"

@fisherwebdev
Copy link
Contributor Author

Are we waiting on something here?

@sophiebits
Copy link
Collaborator

Nope, sorry! No one gets notified when you push commits so it's best to comment back so we know.

sophiebits added a commit that referenced this pull request Aug 28, 2014
[docs] Update Flux TodoMVC Tutorial
@sophiebits sophiebits merged commit 944f49c into facebook:master Aug 28, 2014
sophiebits added a commit that referenced this pull request Sep 3, 2014
[docs] Update Flux TodoMVC Tutorial
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants