Skip to content

Commit

Permalink
[Bug fix] Fix dashboard filters to collect options too (getredash#3759)
Browse files Browse the repository at this point in the history
  • Loading branch information
yoavbls authored and harveyrendell committed Nov 14, 2019
1 parent 9c8698e commit c6d18ac
Showing 1 changed file with 3 additions and 4 deletions.
7 changes: 3 additions & 4 deletions client/app/services/dashboard.js
Original file line number Diff line number Diff line change
Expand Up @@ -19,13 +19,12 @@ export function collectDashboardFilters(dashboard, queryResults, urlParams) {
queryFilter.current = urlParams[queryFilter.name];
}

const filter = { ...queryFilter };
if (!_.has(filters, queryFilter.name)) {
const filter = { ...queryFilter };
filters[filter.name] = filter;
} else {
filters[filter.name].values = _.union(filters[filter.name].values, filter.values);
}

// TODO: merge values. -- maybe - intersect??
// filters[queryFilter.name].originFilters.push(queryFilter);
});
});

Expand Down

0 comments on commit c6d18ac

Please sign in to comment.