Skip to content

Commit

Permalink
FFS-2351: Change calculation of account_count to be correct in filed …
Browse files Browse the repository at this point in the history
…events (#445)

ApplicantSharedIncomeSummary and ApplicantAccessedIncomeSummary events now contain a correct value for account_count.
  • Loading branch information
millerti authored Feb 12, 2025
1 parent 9101ee9 commit 3fd6006
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions app/app/controllers/cbv/summaries_controller.rb
Original file line number Diff line number Diff line change
Expand Up @@ -193,7 +193,7 @@ def track_transmitted_event(cbv_flow, payments)
site_id: cbv_flow.site_id,
cbv_flow_id: cbv_flow.id,
invitation_id: cbv_flow.cbv_flow_invitation_id,
account_count: payments.map { |p| p[:account_id] }.uniq.count,
account_count: cbv_flow.pinwheel_accounts.count,
paystub_count: payments.count,
account_count_with_additional_information:
cbv_flow.additional_information.values.count { |info| info["comment"].present? },
Expand All @@ -210,7 +210,7 @@ def track_accessed_income_summary_event(cbv_flow, payments)
site_id: cbv_flow.site_id,
cbv_flow_id: cbv_flow.id,
invitation_id: cbv_flow.cbv_flow_invitation_id,
account_count: payments.map { |p| p.account_id }.uniq.count,
account_count: cbv_flow.pinwheel_accounts.count,
paystub_count: payments.count,
account_count_with_additional_information:
cbv_flow.additional_information.values.count { |info| info["comment"].present? },
Expand Down

0 comments on commit 3fd6006

Please sign in to comment.