Merge pull request #485 from plotly/n-riesco-patch-1

scheduler: document reason to unset scheduledQuery
plotly · Jul 10, 2018 · adbbbad · adbbbad
2 parents da2609b + e5b8a6c
commit adbbbad
Showing 1 changed file with 3 additions and 0 deletions.
diff --git a/app/components/Settings/Settings.react.js b/app/components/Settings/Settings.react.js
@@ -272,6 +272,9 @@ class Settings extends Component {
     openScheduler() {
       this.setState({ scheduledQuery: this.props.preview.code }, () => {
         this.updateSelectedPanel(2, () => {
+          // Setting scheduledQuery to `null` right after opening the Schedule panel ensures that,
+          // when the user switches to another connector,
+          // `initialCode` isn't set to the `scheduledQuery` used in the previous connector.
           this.setState({ scheduledQuery: null });
         });
       });