Skip to content

Commit

Permalink
Merge pull request #1390 from elementary-data/ele-2410-split-send_ale…
Browse files Browse the repository at this point in the history
…rts-flow-to-multiple-steps

ELE-2410 - split send_alerts flow to multiple steps
  • Loading branch information
IDoneShaveIt authored Jan 28, 2024
2 parents c19d487 + 0ae1895 commit 6594333
Showing 1 changed file with 4 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -262,13 +262,14 @@ def _send_alerts(

# Now update as sent:
self.sent_alert_count = len(sent_successfully_alerts)
self.alerts_api.update_sent_alerts(
[alert.id for alert in sent_successfully_alerts]
)
self._update_sent_alerts([alert.id for alert in sent_successfully_alerts])

# Now update sent alerts counter:
self.execution_properties["sent_alert_count"] = self.sent_alert_count

def _update_sent_alerts(self, alert_ids: List[str]):
self.alerts_api.update_sent_alerts(alert_ids=alert_ids)

def _skip_alerts(self, alerts: List[PendingAlertSchema]):
self.alerts_api.skip_alerts(alerts)

Expand Down

0 comments on commit 6594333

Please sign in to comment.