Merge pull request #12 from IngloriousCoderz/master

xidedix · web-flow · commit 58fed10d97ac · 2018-06-14T16:55:13.000+02:00
Fix switch not updating on props change
diff --git a/src/Switch.js b/src/Switch.js
@@ -63,6 +63,14 @@ class AppSwitch extends Component {
     }
   }
 
+  componentDidUpdate(prevProps) {
+    if (this.props.checked !== prevProps.checked) {
+      this.setState({
+        checked: this.props.checked
+      })
+    }
+  }
+
   render() {
     const { className, disabled, color, name, label, outline, size, required, type, value, dataOn, dataOff, variant, ...attributes } = this.props;
 

Original file line number	Diff line number	Diff line change
`@@ -63,6 +63,14 @@ class AppSwitch extends Component {`
`63`	`63`	`}`
`64`	`64`	`}`
`65`	`65`
	`66`	`+ componentDidUpdate(prevProps) {`
	`67`	`+ if (this.props.checked !== prevProps.checked) {`
	`68`	`+ this.setState({`
	`69`	`+ checked: this.props.checked`
	`70`	`+ })`
	`71`	`+ }`
	`72`	`+ }`
	`73`	`+`
`66`	`74`	`render() {`
`67`	`75`	`const { className, disabled, color, name, label, outline, size, required, type, value, dataOn, dataOff, variant, ...attributes } = this.props;`
`68`	`76`