That could work too. At first I was like hmm is creating new class an overkill.
Perhaps not. Just a bit more typing. and that allows override `.register()`
function + more customization.
```js
class BarPlugin extends ChartPlugin {
constructor(config) {
super({ config, metadata, Chart, transformProps });
}
}
```
vs
```js
const barPlugin = new ChartPlugin({ metadata, Chart, transformProps });
barPlugin.configure(config);
```
[ Full content available at:
https://github.com/apache/incubator-superset/pull/6028 ]
This message was relayed via gitbox.apache.org for [email protected]
