From ddbfedf9e4a9cbf223ecf35eb4d5b8b85b081f20 Mon Sep 17 00:00:00 2001 From: Florrie Date: Wed, 20 Jun 2018 17:04:51 -0300 Subject: [PATCH 1/2] Fix booleans not showing up in monitors --- src/lib/monitor-adapter.js | 12 +++++++++++- 1 file changed, 11 insertions(+), 1 deletion(-) diff --git a/src/lib/monitor-adapter.js b/src/lib/monitor-adapter.js index a1d2237c7..d7bdd304e 100644 --- a/src/lib/monitor-adapter.js +++ b/src/lib/monitor-adapter.js @@ -28,6 +28,16 @@ export default function ({id, spriteName, opcode, params, value}) { if (typeof value === 'number') { value = Number(value.toFixed(6)); } - + + // Turn the value to a string, for handle boolean values + if (typeof value === 'boolean') { + value = value.toString(); + } + + // Lists can contain booleans, which should also be turned to strings + if (Array.isArray(value)) { + value = value.map(item => item.toString()); + } + return {id, label, category, value}; } From 3d5e52a99d641110963d1fada656e75765c82663 Mon Sep 17 00:00:00 2001 From: Florrie Date: Tue, 26 Jun 2018 13:35:22 -0300 Subject: [PATCH 2/2] Fix slider monitors being wonky --- src/containers/slider-monitor.jsx | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/containers/slider-monitor.jsx b/src/containers/slider-monitor.jsx index b6f43eed5..f818a07bd 100644 --- a/src/containers/slider-monitor.jsx +++ b/src/containers/slider-monitor.jsx @@ -15,7 +15,7 @@ class SliderMonitor extends React.Component { ]); this.state = { - value: Number(props.value) + value: props.value }; } componentWillReceiveProps (nextProps) {