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.
This commit is contained in:
@@ -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)
|
||||
|
||||
|
||||
Reference in New Issue
Block a user