fix: skip per-metric rollup check in summary builders
Summary builders fetch all metrics in a single query and don't use params[:metric], so metric_covered? was always returning false and preventing rollup usage for summary reports.
This commit is contained in:
@@ -9,6 +9,12 @@ class V2::Reports::BaseSummaryBuilder
|
||||
|
||||
private
|
||||
|
||||
# Summary builders fetch all metrics in a single query, so the per-metric
|
||||
# check from RollupConditions does not apply.
|
||||
def metric_covered?
|
||||
true
|
||||
end
|
||||
|
||||
def load_data
|
||||
@conversations_count = fetch_conversations_count
|
||||
use_rollup? ? load_rollup_data : load_reporting_events_data
|
||||
|
||||
Reference in New Issue
Block a user