codepic commented on issue #11113: URL: https://github.com/apache/incubator-superset/issues/11113#issuecomment-702018399
@villebro I managed to get the filtering working :+1:  Below are the relevant changes I needed to do in `superset-frontend`  As it seems, the check `form_data.viz_type === 'filter_box'` is hard coded so I added my `viz_type` to the check and my plugin started emitting filter events correctly. There's a catch. If I add `filter_box` into the dashboard, the functionality breaks and the table only listens to the filters set in the `filter_box` plugin. However this is not an issue as I can still use multiple custom plugins which emit filters and they seem to be working as expected:  Since hard coding `viz_type` check is not optimal, I would suggest we would add a flag to the `form_data` when ever a plugin developer adds the `FilterBoxItemControl` into their control panel. Then we can check in `dashboardState.js` and `getInitialState.js` that if this flag is set to `true`. This way plugin developers could start implementing their own filters just by adding `FilterBoxItemControl` into their `controlPanel.ts` and using the `hooks` provided by superset. As it currently stands, having custom filtering functionality is impossible with Superset without modifying `superset-frontend` yourself. ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
