From da3c9527c5836570024e893f6babf685db1c6913 Mon Sep 17 00:00:00 2001 From: Jim Date: Sun, 17 Jan 2016 14:31:27 -0800 Subject: [PATCH] Merge pull request #5872 from SimenB/patch-1 Fix example from #5870 to avoid double rendering (cherry picked from commit c7808cac6946414093136732bc91c2b948af2140) --- docs/tips/12-initial-ajax.md | 20 +++++++++----------- 1 file changed, 9 insertions(+), 11 deletions(-) diff --git a/docs/tips/12-initial-ajax.md b/docs/tips/12-initial-ajax.md index c28835c576..e8d4c960a1 100644 --- a/docs/tips/12-initial-ajax.md +++ b/docs/tips/12-initial-ajax.md @@ -23,19 +23,17 @@ var UserGist = React.createClass({ }, componentDidMount: function() { - this.setState({ - serverRequest: $.get(this.props.source, function(result) { - var lastGist = result[0]; - this.setState({ - username: lastGist.owner.login, - lastGistUrl: lastGist.html_url - }); - }.bind(this)) - }); + this.serverRequest = $.get(this.props.source, function (result) { + var lastGist = result[0]; + this.setState({ + username: lastGist.owner.login, + lastGistUrl: lastGist.html_url + }); + }.bind(this)); }, - + componentWillUnmount: function() { - this.state.serverRequest.abort(); + this.serverRequest.abort(); }, render: function() {