Skip to content

[DOI-2644] Use coalesce for amountsentsubscribers null value#108

Merged
emoralesms merged 1 commit intomainfrom
doi-2644-campaing-ten-status-use-amountsubscriberstosend
Jan 7, 2026
Merged

[DOI-2644] Use coalesce for amountsentsubscribers null value#108
emoralesms merged 1 commit intomainfrom
doi-2644-campaing-ten-status-use-amountsubscriberstosend

Conversation

@emoralesms
Copy link
Contributor

@emoralesms emoralesms commented Jan 7, 2026

Cuando una campaña esta en estado 10 (enviando), no tiene AmountSentSubscribers aún, por lo que el valor de la columna queda en NULL. Con este fix, cuando el valor es NULL usamos momentaneamente AmountSubscribersToSend

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 dbfb134 into main Jan 7, 2026
6 checks passed
@emoralesms emoralesms deleted the doi-2644-campaing-ten-status-use-amountsubscriberstosend branch January 7, 2026 20:12
@fgchaio
Copy link
Contributor

fgchaio commented Jan 7, 2026

👍

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.

2 participants

Comments