Skip to content

Commit

Permalink
ref: fix subscription_id None check (#74640)
Browse files Browse the repository at this point in the history
looks like this might've been skipping some of these as well (always
passing `None` when deleting)

<!-- Describe your PR here. -->
  • Loading branch information
asottile-sentry authored Jul 22, 2024
1 parent 7b5f4ab commit 897d5d1
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/sentry/uptime/subscriptions/tasks.py
Original file line number Diff line number Diff line change
Expand Up @@ -68,7 +68,7 @@ def delete_remote_uptime_subscription(uptime_subscription_id, **kwargs):
subscription.update(subscription_id=None)

if subscription_id is not None:
send_uptime_config_deletion(subscription.subscription_id)
send_uptime_config_deletion(subscription_id)


def send_uptime_subscription_config(subscription: UptimeSubscription) -> str:
Expand Down

0 comments on commit 897d5d1

Please sign in to comment.