Skip to content

Comments

[DOI-2602] Fix dividend case condition#103

Merged
emoralesms merged 1 commit intomainfrom
doi-2602-fix-case-condition
Dec 17, 2025
Merged

[DOI-2602] Fix dividend case condition#103
emoralesms merged 1 commit intomainfrom
doi-2602-fix-case-condition

Conversation

@emoralesms
Copy link
Contributor

@emoralesms emoralesms commented Dec 16, 2025

Estaba mal el case, el dividendo es Subscribers y se estaba usando Sent.
El dividendo de: Open, Not Open y Bounces debe ser el mismo para poder calcular bien los porcentajes y la suma de los 3 de 100%.
Por las dudas que surja la duda, hay que usar Subscribers que corresponde a nuestros enviados: abiertos, no abiertos y rebotes.

Se detecto el problema por unos logs en loggly sobre las cuentas de mmosquera@makingsense.com y llopez+int1_paid1@makingsense.com

Fixes:

Checklist:

  • I have paid attention to this PR title and description
  • I have performed a self-review of my code
  • I have built it locally (or my changes does not affect the build)
  • I have checked all tests still run ok at Doppler.ReportingApiTest project
  • I have added at least one simple unit test covering the new code

@emoralesms emoralesms merged commit 9543f7a into main Dec 17, 2025
5 checks passed
@emoralesms emoralesms deleted the doi-2602-fix-case-condition branch December 17, 2025 11:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants