Caravel dashboard filters

[gbrian]

Fix for #144

clearFiltersremoved. Seems not being used by anyone. Please validate.

[coveralls]

Coverage increased (+0.2%) to 72.365% when pulling 02e111e on gbrian:caravel_dashboard_filters into 866e00d on airbnb:master.

[coveralls]

Coverage increased (+0.2%) to 72.365% when pulling 02e111e on gbrian:caravel_dashboard_filters into 866e00d on airbnb:master.

[landscape-bot]

Code quality remained the same when pulling 02e111e on gbrian:caravel_dashboard_filters into 866e00d on airbnb:master.

[landscape-bot]

Code quality remained the same when pulling 02e111e on gbrian:caravel_dashboard_filters into 866e00d on airbnb:master.

[coveralls]

Coverage increased (+0.03%) to 72.583% when pulling 9384a5d on gbrian:caravel_dashboard_filters into c6da811 on airbnb:master.

[sid88in]

@gbrian any update on this? Does this fix the multiple dynamic filter problems? Can you please rebase with master and open it for testing? I can then try locally.

Thanks

[landscape-bot]

Code quality remained the same when pulling 04f680f on gbrian:caravel_dashboard_filters into 2f64c42 on airbnb:master.

[gbrian]

@sid88in, PR updated.

This tries to fix the issue with multiple filters about loosing filters selection.
I needed to change how filters are store due to the filter behavior present on tables graphs.
So now filters are not based on slices, just in fields. in this way you will be filtering by a field value regardless if you set a filter or click a table value.

Hope this helps.

[coveralls]

Coverage increased (+0.03%) to 81.613% when pulling 04f680f on gbrian:caravel_dashboard_filters into 2f64c42 on airbnb:master.

[sid88in]

Thanks!
I will test this in local and see if it resolves our use case with filter bug.

[coveralls]

Coverage increased (+0.03%) to 82.137% when pulling a520245 on gbrian:caravel_dashboard_filters into 087c47a on airbnb:master.

[mistercrunch]

This got tackled somewhere else. Thanks for the PR though!