Skip to content

Commit 40ba0d0

Browse files
authored
🔧 chore: use IntegrationProviderSlug.Opsgenie instead of magic str (#95165)
1 parent eb42cac commit 40ba0d0

File tree

3 files changed

+7
-4
lines changed

3 files changed

+7
-4
lines changed

src/sentry/integrations/opsgenie/actions/form.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,6 +11,7 @@
1111
from sentry.integrations.opsgenie.utils import get_team
1212
from sentry.integrations.services.integration import integration_service
1313
from sentry.integrations.services.integration.model import RpcOrganizationIntegration
14+
from sentry.integrations.types import IntegrationProviderSlug
1415
from sentry.utils.forms import set_field_choices
1516

1617
INVALID_TEAM = 1
@@ -69,7 +70,7 @@ def _validate_team(self, team_id: str | None, integration_id: int | None) -> Non
6970
"team": dict(self._teams).get(team_id),
7071
}
7172
integration = integration_service.get_integration(
72-
integration_id=integration_id, provider="opsgenie"
73+
integration_id=integration_id, provider=IntegrationProviderSlug.OPSGENIE.value
7374
)
7475
org_integration = integration_service.get_organization_integration(
7576
integration_id=integration_id,

src/sentry/integrations/opsgenie/actions/notification.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,7 @@
1313
)
1414
from sentry.integrations.opsgenie.utils import get_team
1515
from sentry.integrations.services.integration import integration_service
16+
from sentry.integrations.types import IntegrationProviderSlug
1617
from sentry.rules.actions import IntegrationEventAction
1718
from sentry.shared_integrations.exceptions import ApiError
1819

@@ -25,7 +26,7 @@ class OpsgenieNotifyTeamAction(IntegrationEventAction):
2526
"Send a notification to Opsgenie account {account} and team {team} with {priority} priority"
2627
)
2728
prompt = "Send an Opsgenie notification"
28-
provider = "opsgenie"
29+
provider = IntegrationProviderSlug.OPSGENIE.value
2930
integration_key = "account"
3031

3132
def __init__(self, *args, **kwargs):

src/sentry/integrations/opsgenie/client.py

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,7 @@
99
from sentry.integrations.on_call.metrics import OnCallInteractionType
1010
from sentry.integrations.opsgenie.metrics import record_event, record_lifecycle_termination_level
1111
from sentry.integrations.services.integration.model import RpcIntegration
12+
from sentry.integrations.types import IntegrationProviderSlug
1213
from sentry.models.group import Group
1314
from sentry.notifications.utils.links import create_link_to_workflow
1415
from sentry.notifications.utils.rules import get_key_from_rule_data
@@ -22,7 +23,7 @@
2223

2324

2425
class OpsgenieClient(ApiClient):
25-
integration_name = "opsgenie"
26+
integration_name = IntegrationProviderSlug.OPSGENIE.value
2627

2728
def __init__(self, integration: RpcIntegration | Integration, integration_key: str) -> None:
2829
self.integration = integration
@@ -86,7 +87,7 @@ def build_issue_alert_payload(
8687
if group:
8788
payload["alias"] = f"sentry: {group.id}"
8889
payload["entity"] = group.culprit if group.culprit else ""
89-
group_params = {"referrer": "opsgenie"}
90+
group_params = {"referrer": IntegrationProviderSlug.OPSGENIE.value}
9091
if notification_uuid:
9192
group_params["notification_uuid"] = notification_uuid
9293
rule_workflow_context = {}

0 commit comments

Comments
 (0)