[GitHub] jaylindquist commented on issue #5269: Filters as a native dashboard (v2) construct
jaylindquist commented on issue #5269: Filters as a native dashboard (v2) construct URL: https://github.com/apache/incubator-superset/issues/5269#issuecomment-445968034 Due to our changing priorities, we have been unable to complete the dashboard level filters feature. You can find our in progress work on my feature branch (https://github.com/jaylindquist/incubator-superset/tree/feature/native-filter) and I can create a PR if you are interested in keeping what we've been able to start. Completed tasks: * Add a drag-and-drop filters container at the top of dashboards * Add filters to the filter container (filters are added, but configuration is hard coded) * Remove filters from container * Remove filters container from dashboard * Update Dashboard short links to create `preselect_filters` parameter without chart IDs. URL short links are now generated as `{ "column1": ["val1"], "column2": ["val2", "val3"] }` * Update `preselect_filters` parameter to support filters without chart IDs while being backwards compatible. `preselect_filters` now supports the above format as well as the existing format `{ "chartId1": { "column1": ["val1"]}, "chartId2": { "column2": ["val2", "val3"]}}` The following shows adding filters to an existing dashboard: ![superset-filters](https://user-images.githubusercontent.com/222699/49760761-e1df6300-fc8a-11e8-9e97-c05ce4206cbf.gif) There is still work to be done: * Creating a modal to add new filters * Manage date filters for each datasource * Global filter configuration (for example, instant filtering) * Saving selected filter options when the dashboard is saved If you would like me to create a PR for this work, let me know. I apologize for not being able to complete the feature, but other tasks took priority for us. This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services - To unsubscribe, e-mail: notifications-unsubscr...@superset.apache.org For additional commands, e-mail: notifications-h...@superset.apache.org
[GitHub] jaylindquist commented on issue #5269: Filters as a native dashboard (v2) construct
jaylindquist commented on issue #5269: Filters as a native dashboard (v2) construct URL: https://github.com/apache/incubator-superset/issues/5269#issuecomment-433407446 I apologize for the lack of updates on this. Our team hit a series of emergencies that took priority over the filter changes. We've only now been able to start the work. Changes will be going into my forked at https://github.com/jaylindquist/incubator-superset/tree/feature/native-filter This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services - To unsubscribe, e-mail: notifications-unsubscr...@superset.apache.org For additional commands, e-mail: notifications-h...@superset.apache.org
[GitHub] jaylindquist commented on issue #5269: Filters as a native dashboard (v2) construct
jaylindquist commented on issue #5269: Filters as a native dashboard (v2) construct URL: https://github.com/apache/incubator-superset/issues/5269#issuecomment-412535393 @mistercrunch @graceguo-supercat Just pinging to see if there is any other feedback before we start planning our implementation This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services - To unsubscribe, e-mail: notifications-unsubscr...@superset.apache.org For additional commands, e-mail: notifications-h...@superset.apache.org
[GitHub] jaylindquist commented on issue #5269: Filters as a native dashboard (v2) construct
jaylindquist commented on issue #5269: Filters as a native dashboard (v2) construct URL: https://github.com/apache/incubator-superset/issues/5269#issuecomment-412109988 @graceguo-supercat Today superset has a "Time Column" for filters that it uses to manage which column to filter on. I'm not sure how a generic time filter would work. For instance, a table of events that have a start_date column and end_date column. If a user wanted to filter on all events starting in January, they wouldn't want the filter to apply to the end_date column as well. They would need to specify which column(s) the time range should apply to. This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services - To unsubscribe, e-mail: notifications-unsubscr...@superset.apache.org For additional commands, e-mail: notifications-h...@superset.apache.org
[GitHub] jaylindquist commented on issue #5269: Filters as a native dashboard (v2) construct
jaylindquist commented on issue #5269: Filters as a native dashboard (v2) construct URL: https://github.com/apache/incubator-superset/issues/5269#issuecomment-411389249 @mistercrunch We would like to add a proposal for the native dashboard filter as outlined in the first post. We are looking for feedback and approval before implementing. ### Dashboard changes * Global filters will be the focus of the MVP * Tab and section filters would work similarly but will not be included in the initial implementation. The initial focus will be to replace the existing Filter Box chart with a native, global dashboard filter * The filter component is drag & drop from the insert panel, similar to tabs * Global filters go above dashboard-level tabs and below the dashboard title (see images below for an example) * Tab filters go below tabs * Section filters go at the top of the section * New filters can be added when the dashboard is in "Edit Mode" via a button that appears after the list of filters * Individual filters are added based on datasource / column name * Clicking the New Filter button will open a modal with "Data Source" and "Column" options to select the filter column * If the column selected is a time column, the 'Since' / 'Until' filters will be added * Filter controls for the dashboard are initially collapsed and can be expanded / collapsed as needed (see images below for an example) * Additional options such as timegrain and instant filtering are added via the configuration cog ### Chart changes * Filter box will be removed as a chart type * Filter boxes that are currently included in dashboards will need to be removed from the dashboard and options will be added as global filter options * All existing filter box charts are deleted (not sure what else can be done with them since they can't be added to dashboards anymore) ### URL changes * The `preselect_filters` query parameter can still be used to fill in filters via a link. It is currently a map which includes the Filter Box ID. The format will need to change into a map of column name to arrays instead * Old format ```json { "414": { "column1": [ "value1", "value2" ] }, "12": { "column2": [ "value3" ] } } ``` * New format ```json { "column1": [ "value1", "value2" ], "column2": [ "value3" ] } ``` * This change can be made backwards compatible by collecting the filters in the old format, ignoring the ID, and applying the filter values based on the column name ### Dashboard mockups *Collapsed filters* ![global filter closed](https://user-images.githubusercontent.com/222699/43837185-59133bc8-9add-11e8-895e-02c66898796a.png) *Opened filters* ![global filter open](https://user-images.githubusercontent.com/222699/43837195-60e4c4fc-9add-11e8-9f8d-6d2f6c914470.png) *New filter dialog* ![filters opened edit edit filter](https://user-images.githubusercontent.com/222699/43837207-66c6d61c-9add-11e8-8b9d-8df0ee8b40a9.png) This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services - To unsubscribe, e-mail: notifications-unsubscr...@superset.apache.org For additional commands, e-mail: notifications-h...@superset.apache.org