Skip to content

Commit 686daa2

Browse files
authored
ref(messaging): Updated Integration Types Imports (#72262)
Final PR to update the rest of the integration imports in sentry over to the new module that I created. I edit 1 `__init__.py` file, plan is to clean it up completely in a followup pr.
1 parent 7b8596a commit 686daa2

File tree

111 files changed

+120
-116
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

111 files changed

+120
-116
lines changed

src/sentry/api/bases/external_actor.py

+2-1
Original file line numberDiff line numberDiff line change
@@ -16,13 +16,14 @@
1616
validate_integration_id,
1717
)
1818
from sentry.api.validators.integrations import validate_provider
19+
from sentry.integrations.types import ExternalProviders
20+
from sentry.integrations.utils.providers import get_provider_choices
1921
from sentry.models.integrations.external_actor import ExternalActor
2022
from sentry.models.organization import Organization
2123
from sentry.models.team import Team
2224
from sentry.services.hybrid_cloud.organization import organization_service
2325
from sentry.services.hybrid_cloud.user import RpcUser
2426
from sentry.services.hybrid_cloud.user.service import user_service
25-
from sentry.types.integrations import ExternalProviders, get_provider_choices
2627

2728
AVAILABLE_PROVIDERS = {
2829
ExternalProviders.GITHUB,

src/sentry/api/endpoints/user_notification_settings_providers.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -10,12 +10,12 @@
1010
from sentry.api.exceptions import ParameterValidationError
1111
from sentry.api.serializers import serialize
1212
from sentry.api.validators.notifications import validate_type
13+
from sentry.integrations.types import PERSONAL_NOTIFICATION_PROVIDERS
1314
from sentry.models.notificationsettingprovider import NotificationSettingProvider
1415
from sentry.models.user import User
1516
from sentry.notifications.serializers import NotificationSettingsProviderSerializer
1617
from sentry.notifications.types import NotificationSettingsOptionEnum
1718
from sentry.notifications.validators import UserNotificationSettingsProvidersDetailsSerializer
18-
from sentry.types.integrations import PERSONAL_NOTIFICATION_PROVIDERS
1919

2020

2121
@control_silo_endpoint

src/sentry/api/serializers/models/external_actor.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -2,9 +2,9 @@
22
from typing import Any, TypedDict
33

44
from sentry.api.serializers import Serializer, register
5+
from sentry.integrations.utils.providers import get_provider_string
56
from sentry.models.integrations.external_actor import ExternalActor
67
from sentry.models.user import User
7-
from sentry.types.integrations import get_provider_string
88

99

1010
class ExternalActorResponseOptional(TypedDict, total=False):

src/sentry/api/validators/external_actor.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -4,9 +4,9 @@
44

55
from sentry.api.exceptions import ParameterValidationError
66
from sentry.api.validators.integrations import validate_provider
7+
from sentry.integrations.types import ExternalProviders
78
from sentry.models.organization import Organization
89
from sentry.services.hybrid_cloud.integration import integration_service
9-
from sentry.types.integrations import ExternalProviders
1010

1111
EXTERNAL_ID_LENGTH_MIN = 1
1212
EXTERNAL_ID_LENGTH_MAX = 64

src/sentry/api/validators/integrations.py

+2-1
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
from sentry.api.exceptions import ParameterValidationError
2-
from sentry.types.integrations import ExternalProviders, get_provider_enum
2+
from sentry.integrations.types import ExternalProviders
3+
from sentry.integrations.utils.providers import get_provider_enum
34

45

56
def validate_provider(

src/sentry/api/validators/project_codeowners.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -5,14 +5,14 @@
55

66
from django.db.models import Subquery
77

8+
from sentry.integrations.types import ExternalProviders
89
from sentry.models.integrations.external_actor import ExternalActor
910
from sentry.models.organizationmember import OrganizationMember
1011
from sentry.models.organizationmemberteam import OrganizationMemberTeam
1112
from sentry.models.project import Project
1213
from sentry.models.team import Team
1314
from sentry.ownership.grammar import parse_code_owners
1415
from sentry.services.hybrid_cloud.user.service import user_service
15-
from sentry.types.integrations import ExternalProviders
1616

1717

1818
def validate_association_emails(

src/sentry/digests/utils.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,7 @@
99

1010
from sentry.digests import Digest, Record
1111
from sentry.eventstore.models import Event
12+
from sentry.integrations.types import ExternalProviders
1213
from sentry.models.group import Group
1314
from sentry.models.project import Project
1415
from sentry.models.projectownership import ProjectOwnership
@@ -17,7 +18,6 @@
1718
from sentry.notifications.types import ActionTargetType, FallthroughChoiceType
1819
from sentry.notifications.utils.participants import get_send_to
1920
from sentry.types.actor import Actor
20-
from sentry.types.integrations import ExternalProviders
2121

2222

2323
def get_digest_metadata(

src/sentry/incidents/action_handlers.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,7 @@
1616
from sentry.incidents.charts import build_metric_alert_chart
1717
from sentry.incidents.models.alert_rule import AlertRuleThresholdType, AlertRuleTriggerAction
1818
from sentry.incidents.models.incident import INCIDENT_STATUS, IncidentStatus, TriggerStatus
19+
from sentry.integrations.types import ExternalProviders
1920
from sentry.models.rulesnooze import RuleSnooze
2021
from sentry.models.user import User
2122
from sentry.notifications.types import NotificationSettingEnum
@@ -25,7 +26,6 @@
2526
from sentry.services.hybrid_cloud.user_option import RpcUserOption, user_option_service
2627
from sentry.snuba.metrics import format_mri_field, is_mri_field
2728
from sentry.types.actor import Actor, ActorType
28-
from sentry.types.integrations import ExternalProviders
2929
from sentry.utils.email import MessageBuilder, get_email_addresses
3030

3131

src/sentry/integrations/discord/message_builder/issues.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -17,11 +17,11 @@
1717
get_color,
1818
get_title_link,
1919
)
20+
from sentry.integrations.types import ExternalProviders
2021
from sentry.models.group import Group, GroupStatus
2122
from sentry.models.project import Project
2223
from sentry.models.rule import Rule
2324
from sentry.notifications.notifications.base import ProjectNotification
24-
from sentry.types.integrations import ExternalProviders
2525

2626
from ..message_builder.base.component import DiscordComponentCustomIds as CustomIds
2727

src/sentry/integrations/discord/views/link_identity.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -5,11 +5,11 @@
55
from django.views.decorators.cache import never_cache
66

77
from sentry import analytics
8+
from sentry.integrations.types import ExternalProviders
89
from sentry.integrations.utils.identities import get_identity_or_404
910
from sentry.models.identity import Identity
1011
from sentry.services.hybrid_cloud.integration.model import RpcIntegration
1112
from sentry.types.actor import ActorType
12-
from sentry.types.integrations import ExternalProviders
1313
from sentry.utils.http import absolute_uri
1414
from sentry.utils.signing import sign, unsign
1515
from sentry.web.frontend.base import BaseView, control_silo_view

src/sentry/integrations/discord/views/unlink_identity.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -6,11 +6,11 @@
66
from django.views.decorators.cache import never_cache
77

88
from sentry import analytics
9+
from sentry.integrations.types import ExternalProviders
910
from sentry.integrations.utils.identities import get_identity_or_404
1011
from sentry.models.identity import Identity
1112
from sentry.services.hybrid_cloud.integration.model import RpcIntegration
1213
from sentry.types.actor import ActorType
13-
from sentry.types.integrations import ExternalProviders
1414
from sentry.utils.http import absolute_uri
1515
from sentry.utils.signing import sign, unsign
1616
from sentry.web.frontend.base import BaseView, control_silo_view

src/sentry/integrations/github/client.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,7 @@
1717
)
1818
from sentry.integrations.github.utils import get_jwt, get_next_link
1919
from sentry.integrations.mixins.commit_context import FileBlameInfo, SourceLineInfo
20+
from sentry.integrations.types import EXTERNAL_PROVIDERS, ExternalProviders
2021
from sentry.integrations.utils.code_mapping import (
2122
MAX_CONNECTION_ERRORS,
2223
Repo,
@@ -31,7 +32,6 @@
3132
from sentry.shared_integrations.client.proxy import IntegrationProxyClient
3233
from sentry.shared_integrations.exceptions import ApiError, ApiRateLimitedError
3334
from sentry.shared_integrations.response.mapping import MappingApiResponse
34-
from sentry.types.integrations import EXTERNAL_PROVIDERS, ExternalProviders
3535
from sentry.utils import metrics
3636
from sentry.utils.cache import cache
3737

src/sentry/integrations/issue_alert_image_builder.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -9,14 +9,14 @@
99
get_approx_start_time,
1010
get_relative_time,
1111
)
12+
from sentry.integrations.types import ExternalProviderEnum
1213
from sentry.issues.grouptype import (
1314
PerformanceP95EndpointRegressionGroupType,
1415
ProfileFunctionRegressionType,
1516
)
1617
from sentry.models.apikey import ApiKey
1718
from sentry.models.group import Group
1819
from sentry.snuba.referrer import Referrer
19-
from sentry.types.integrations import ExternalProviderEnum
2020
from sentry.utils import metrics
2121
from sentry.utils.performance_issues.detectors.utils import escape_transaction
2222

src/sentry/integrations/jira/integration.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,7 @@
3737
IntegrationError,
3838
IntegrationFormError,
3939
)
40-
from sentry.tasks.integrations import migrate_issues
40+
from sentry.tasks.integrations.migrate_issues import migrate_issues
4141
from sentry.utils.strings import truncatechars
4242

4343
from .client import JiraCloudClient

src/sentry/integrations/jira_server/integration.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -44,7 +44,7 @@
4444
IntegrationError,
4545
IntegrationFormError,
4646
)
47-
from sentry.tasks.integrations import migrate_issues
47+
from sentry.tasks.integrations.migrate_issues import migrate_issues
4848
from sentry.utils.hashlib import sha1_text
4949
from sentry.utils.http import absolute_uri
5050
from sentry.web.helpers import render_to_response

src/sentry/integrations/message_builder.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,7 @@
66
from sentry import features
77
from sentry.eventstore.models import GroupEvent
88
from sentry.integrations.slack.message_builder import LEVEL_TO_COLOR, SLACK_URL_FORMAT
9+
from sentry.integrations.types import EXTERNAL_PROVIDERS, ExternalProviders
910
from sentry.issues.grouptype import GroupCategory
1011
from sentry.models.environment import Environment
1112
from sentry.models.group import Group
@@ -15,7 +16,6 @@
1516
from sentry.notifications.notifications.base import BaseNotification
1617
from sentry.notifications.notifications.rules import AlertRuleNotification
1718
from sentry.services.hybrid_cloud.user import RpcUser
18-
from sentry.types.integrations import EXTERNAL_PROVIDERS, ExternalProviders
1919
from sentry.utils.http import absolute_uri
2020

2121

src/sentry/integrations/msteams/card_builder/notifications.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -11,11 +11,11 @@
1111
from sentry.integrations.msteams.card_builder import MSTEAMS_URL_FORMAT
1212
from sentry.integrations.msteams.card_builder.base import MSTeamsMessageBuilder
1313
from sentry.integrations.msteams.card_builder.block import OpenUrlAction
14+
from sentry.integrations.types import ExternalProviders
1415
from sentry.notifications.notifications.activity.base import GroupActivityNotification
1516
from sentry.notifications.notifications.base import BaseNotification
1617
from sentry.notifications.utils.actions import MessageAction
1718
from sentry.types.actor import Actor
18-
from sentry.types.integrations import ExternalProviders
1919

2020
from .block import (
2121
Action,

src/sentry/integrations/msteams/link_identity.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -5,9 +5,9 @@
55
from django.views.decorators.cache import never_cache
66
from rest_framework.request import Request
77

8+
from sentry.integrations.types import ExternalProviders
89
from sentry.integrations.utils import get_identity_or_404
910
from sentry.models.identity import Identity
10-
from sentry.types.integrations import ExternalProviders
1111
from sentry.utils.http import absolute_uri
1212
from sentry.utils.signing import sign, unsign
1313
from sentry.web.frontend.base import BaseView, control_silo_view

src/sentry/integrations/msteams/notifications.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,7 @@
99
from sentry.integrations.msteams.card_builder.block import AdaptiveCard
1010
from sentry.integrations.msteams.utils import get_user_conversation_id
1111
from sentry.integrations.notifications import get_context, get_integrations_by_channel_by_recipient
12+
from sentry.integrations.types import ExternalProviders
1213
from sentry.models.team import Team
1314
from sentry.models.user import User
1415
from sentry.notifications.notifications.activity.assigned import AssignedActivityNotification
@@ -25,7 +26,6 @@
2526
from sentry.notifications.notifications.rules import AlertRuleNotification
2627
from sentry.notifications.notify import register_notification_provider
2728
from sentry.types.actor import Actor
28-
from sentry.types.integrations import ExternalProviders
2929
from sentry.utils import metrics
3030

3131
from .card_builder.notifications import (

src/sentry/integrations/notifications.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,7 @@
55
from typing import Any
66

77
from sentry.constants import ObjectStatus
8+
from sentry.integrations.types import EXTERNAL_PROVIDERS, ExternalProviders
89
from sentry.models.integrations.external_actor import ExternalActor
910
from sentry.models.organization import Organization
1011
from sentry.models.team import Team
@@ -13,7 +14,6 @@
1314
from sentry.services.hybrid_cloud.integration import RpcIntegration, integration_service
1415
from sentry.services.hybrid_cloud.user import RpcUser
1516
from sentry.types.actor import Actor
16-
from sentry.types.integrations import EXTERNAL_PROVIDERS, ExternalProviders
1717

1818

1919
def get_context(

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

+1-1
Original file line numberDiff line numberDiff line change
@@ -6,8 +6,8 @@
66
from django import forms
77
from django.utils.translation import gettext_lazy as _
88

9+
from sentry.integrations.types import ExternalProviders
910
from sentry.services.hybrid_cloud.integration import integration_service
10-
from sentry.types.integrations import ExternalProviders
1111

1212

1313
def _validate_int_field(field: str, cleaned_data: Mapping[str, Any]) -> int | None:

src/sentry/integrations/slack/client.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -9,10 +9,10 @@
99
from sentry.constants import ObjectStatus
1010
from sentry.db.postgres.transactions import in_test_hide_transaction_boundary
1111
from sentry.integrations.client import ApiClient
12+
from sentry.integrations.types import EXTERNAL_PROVIDERS, ExternalProviders
1213
from sentry.services.hybrid_cloud.integration import integration_service
1314
from sentry.shared_integrations.client import BaseApiResponse
1415
from sentry.shared_integrations.exceptions import ApiError
15-
from sentry.types.integrations import EXTERNAL_PROVIDERS, ExternalProviders
1616
from sentry.utils import json, metrics
1717

1818
SLACK_DATADOG_METRIC = "integrations.slack.http_response"

src/sentry/integrations/slack/message_builder/image_block_builder.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -2,12 +2,12 @@
22
from sentry.integrations.issue_alert_image_builder import IssueAlertImageBuilder
33
from sentry.integrations.slack.message_builder import SlackBlock
44
from sentry.integrations.slack.message_builder.base.block import BlockSlackMessageBuilder
5+
from sentry.integrations.types import ExternalProviderEnum
56
from sentry.issues.grouptype import (
67
PerformanceP95EndpointRegressionGroupType,
78
ProfileFunctionRegressionType,
89
)
910
from sentry.models.group import Group
10-
from sentry.types.integrations import ExternalProviderEnum
1111

1212
IMAGE_ALT = {
1313
PerformanceP95EndpointRegressionGroupType: "P95(transaction.duration)",

src/sentry/integrations/slack/message_builder/issues.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -34,6 +34,7 @@
3434
from sentry.integrations.slack.message_builder.image_block_builder import ImageBlockBuilder
3535
from sentry.integrations.slack.message_builder.time_utils import get_approx_start_time, time_since
3636
from sentry.integrations.slack.utils.escape import escape_slack_markdown_text, escape_slack_text
37+
from sentry.integrations.types import ExternalProviders
3738
from sentry.issues.grouptype import (
3839
GroupCategory,
3940
PerformanceP95EndpointRegressionGroupType,
@@ -59,7 +60,6 @@
5960
from sentry.snuba.referrer import Referrer
6061
from sentry.types.actor import Actor
6162
from sentry.types.group import SUBSTATUS_TO_STR
62-
from sentry.types.integrations import ExternalProviders
6363

6464
STATUSES = {"resolved": "resolved", "ignored": "ignored", "unresolved": "re-opened"}
6565
SUPPORTED_COMMIT_PROVIDERS = (

src/sentry/integrations/slack/message_builder/notifications/base.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -8,9 +8,9 @@
88
from sentry.integrations.slack.message_builder import SlackBlock
99
from sentry.integrations.slack.message_builder.base.block import BlockSlackMessageBuilder
1010
from sentry.integrations.slack.utils.escape import escape_slack_text
11+
from sentry.integrations.types import ExternalProviders
1112
from sentry.notifications.notifications.base import BaseNotification
1213
from sentry.types.actor import Actor
13-
from sentry.types.integrations import ExternalProviders
1414

1515

1616
class SlackNotificationsMessageBuilder(BlockSlackMessageBuilder):

src/sentry/integrations/slack/message_builder/notifications/daily_summary.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -11,12 +11,12 @@
1111
from sentry.integrations.message_builder import build_attachment_text, build_attachment_title
1212
from sentry.integrations.slack.message_builder import SlackBlock
1313
from sentry.integrations.slack.utils.escape import escape_slack_text
14+
from sentry.integrations.types import ExternalProviders
1415
from sentry.models.project import Project
1516
from sentry.models.release import Release
1617
from sentry.notifications.notifications.base import BaseNotification
1718
from sentry.tasks.summaries.utils import COMPARISON_PERIOD
1819
from sentry.types.actor import Actor
19-
from sentry.types.integrations import ExternalProviders
2020
from sentry.utils.http import absolute_uri
2121

2222
from .base import SlackNotificationsMessageBuilder

src/sentry/integrations/slack/notifications.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,7 @@
1313
from sentry.integrations.slack.message_builder import SlackBlock
1414
from sentry.integrations.slack.message_builder.base.block import BlockSlackMessageBuilder
1515
from sentry.integrations.slack.message_builder.notifications import get_message_builder
16+
from sentry.integrations.types import ExternalProviders
1617
from sentry.models.integrations.integration import Integration
1718
from sentry.notifications.additional_attachment_manager import get_additional_attachment
1819
from sentry.notifications.notifications.base import BaseNotification
@@ -21,7 +22,6 @@
2122
from sentry.silo.base import SiloMode
2223
from sentry.tasks.integrations.slack import post_message, post_message_control
2324
from sentry.types.actor import Actor
24-
from sentry.types.integrations import ExternalProviders
2525
from sentry.utils import metrics
2626

2727
logger = logging.getLogger("sentry.notifications")

src/sentry/integrations/slack/service.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,7 @@
1515
AssignedActivityNotification,
1616
ExternalIssueCreatedActivityNotification,
1717
)
18+
from sentry.integrations.types import ExternalProviderEnum
1819
from sentry.integrations.utils.common import get_active_integration_for_organization
1920
from sentry.models.activity import Activity
2021
from sentry.models.options.organization_option import OrganizationOption
@@ -31,7 +32,6 @@
3132
from sentry.notifications.notifications.activity.unassigned import UnassignedActivityNotification
3233
from sentry.notifications.notifications.activity.unresolved import UnresolvedActivityNotification
3334
from sentry.types.activity import ActivityType
34-
from sentry.types.integrations import ExternalProviderEnum
3535

3636
_default_logger = getLogger(__name__)
3737

src/sentry/integrations/slack/views/link_identity.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -3,12 +3,12 @@
33
from django.utils.decorators import method_decorator
44
from rest_framework.request import Request
55

6+
from sentry.integrations.types import ExternalProviderEnum, ExternalProviders
67
from sentry.integrations.utils import get_identity_or_404
78
from sentry.models.identity import Identity
89
from sentry.notifications.notificationcontroller import NotificationController
910
from sentry.notifications.notifications.integration_nudge import IntegrationNudgeNotification
1011
from sentry.services.hybrid_cloud.integration.model import RpcIntegration
11-
from sentry.types.integrations import ExternalProviderEnum, ExternalProviders
1212
from sentry.utils.signing import unsign
1313
from sentry.web.frontend.base import BaseView, control_silo_view
1414
from sentry.web.helpers import render_to_response

0 commit comments

Comments
 (0)