Can't pass null to Charts children

[brandly]

🐛Bug report

Describe the bug

I want to do this:

<Charts>
  {!hasDuplicateTimestamps && (
    <LineChart
      axis={key}
      series={series}
      columns={columns}
      style={style}
    />
  )}
  <ScatterChart
    axis={key}
    series={series}
    columns={columns}
    style={style}
  />
</Charts>

I might have multiple events at a given timestamp. If I do, I don't want to render a line.

Most places where children are looped over, nulls are checked:

react-timeseries-charts/src/components/ChartRow.js

Lines 312 to 313 in aa9c9b3

	React.Children.forEach(this.props.children, child => {
	if (child === null) return;

but there's no check here:

react-timeseries-charts/src/components/ChartRow.js

Lines 316 to 317 in aa9c9b3

	React.Children.forEach(charts.props.children, chart => {
	if (!_.has(chart.props, "visible") || chart.props.visible) {

resulting in this stack trace:

Uncaught (in promise) TypeError: Cannot read property 'props' of null
    at ChartRow.js:377
    at forEachSingleChild (react.development.js:1118)
    at traverseAllChildrenImpl (react.development.js:1007)
    ...

To Reproduce
Steps to reproduce the behavior:

1. Pass a null into the children for a <Charts> instance
2. View crash

Expected behavior
nulls are handle. I believe it's just a matter of adding if (chart === null) return; to line 317 of ChartRow.js

---

want me to send a PR?

[f3rmat]

Had this issue recently. One workaround is to use the visible prop.