Skip to content

[DOI-2639] Add conditions#107

Merged
emoralesms merged 1 commit intomainfrom
doi-2639-add-same-conditions
Jan 7, 2026
Merged

[DOI-2639] Add conditions#107
emoralesms merged 1 commit intomainfrom
doi-2639-add-same-conditions

Conversation

@emoralesms
Copy link
Contributor

@emoralesms emoralesms commented Jan 7, 2026

Faltaba agregar en la parte de subscriber las mismas condiciones que habia para campaña, ya que encontramos el siguiente escenario:

image

Donde una campaña estaba eliminada pero tenia spam, por lo que la mostraba en el dashboard.

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 requested a review from fgchaio January 7, 2026 19:14
@emoralesms emoralesms enabled auto-merge January 7, 2026 19:15
@emoralesms emoralesms merged commit b632e7e into main Jan 7, 2026
5 checks passed
@emoralesms emoralesms deleted the doi-2639-add-same-conditions branch January 7, 2026 19:17
@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