From af59acd6a9468d8e4eb545dbdf4442f2caf75be5 Mon Sep 17 00:00:00 2001 From: Shivam Mishra Date: Wed, 11 Feb 2026 22:52:12 +0530 Subject: [PATCH] docs: Add TODO to remove use_rollup override before merge Add TODO comment noting that the use_rollup parameter override is only for validation purposes via the comparison rake task and should be removed before merging to production. --- app/builders/v2/reports/base_summary_builder.rb | 1 + 1 file changed, 1 insertion(+) diff --git a/app/builders/v2/reports/base_summary_builder.rb b/app/builders/v2/reports/base_summary_builder.rb index 968c9bc02..12475afbb 100644 --- a/app/builders/v2/reports/base_summary_builder.rb +++ b/app/builders/v2/reports/base_summary_builder.rb @@ -10,6 +10,7 @@ class V2::Reports::BaseSummaryBuilder private def use_rollup? + # TODO: Remove this override before merging. This is only for validation via comparison rake task. # If use_rollup is explicitly set in params, respect that return params[:use_rollup] if params.key?(:use_rollup)