fix: refresh cache memory leak example (#784)

* docs(RefreshCache): fix guide memory leak * fix(RefreshCache): fix memory leak
algolia · Jan 2, 2018 · cf228ac · cf228ac
1 parent 349bf21
commit cf228ac
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 2 deletions.
diff --git a/docgen/src/guide/Refreshing_cache.md b/docgen/src/guide/Refreshing_cache.md
@@ -82,7 +82,11 @@ class App extends Component {
   }
 
   componentDidMount() {
-    setInterval(() => this.setState({ refresh: true }), 5000);
+    this.interval = setInterval(() => this.setState({ refresh: true }), 5000);
+  }
+
+  componentWillUnmount() {
+    clearInterval(this.interval);
   }
 
   onSearchStateChange = () => {

diff --git a/stories/RefreshCache.stories.js b/stories/RefreshCache.stories.js
@@ -137,7 +137,7 @@ class App extends Component {
   }
 
   componentDidMount() {
-    setInterval(
+    this.interval = setInterval(
       () =>
         this.setState(prevState => ({
           refresh: prevState.count === 5,
@@ -147,6 +147,10 @@ class App extends Component {
     );
   }
 
+  componentWillUnmount() {
+    clearInterval(this.interval);
+  }
+
   render() {
     return (
       <InstantSearch