add 'cache: false,' to tutorial

[brigand]

Re: React tutorial - ajax cache headsup (Google Groups).

[ToucheSir]

Wouldn't it be simpler to disable caching in Firefox's devtools? Setting cache: false might give people the wrong idea regarding using asynchronous requests with React.

[brigand]

@ToucheSir it's probably simpler to sneak it into the code than to ask/expect people to do something special in their browser. It already tripped one person up, which usually means at least 4 or 5 others had the same problem and didn't report it.

[zpao]

Let's add an inline comment about why we're using cache: false to help educate the copy+pasters.

[sophiebits]

Also note that this appears 5 times in the tutorial and the line highlights are messed up by adding a line.

[zpao]

@brigand, are you still interested in doing this?

[brigand]

@zpao doing what? I personally don't think an explanation is needed; it'd just distract from the point of the tutorial.

Sorry for the late response.

[zpao]

I'm asking if you're interested in following through with this PR - there are unresolved issues in the comments from me and @spicyj. I still think we should add a short comment on the first use.

[brigand]

I added the inline comment,

cache: false, // needed for SimpleHTTPServer

and put the other cache: false lines in.

[zpao]

Last couple weeks have been busy, I apologize for the really sporadic pings here :/

Can we get the line highlights fixed like @spicyj mentioned above? We use a syntax for codeblocks which results in specific lines being highlighted and just inserting new lines without adjusting offsets results in bad highlighting.

If you're not up for it, I understand, just let me know!

Before:

After:

[zpao]

With using real servers now in https://github.com/reactjs/react-tutorial, I think we can just skip this.