Skip to content

feat(scm): Add SCM subscriptions platform publisher #107441

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Draft
cmanallen wants to merge 42 commits into
base: cmanallen/scm-platform
Choose a base branch
from
Draft

feat(scm): Add SCM subscriptions platform publisher #107441

cmanallen wants to merge 42 commits into from

Conversation

@cmanallen
Copy link
Member

@cmanallen cmanallen commented Feb 2, 2026 •
edited
Loading

No description provided.

@cmanallen cmanallen changed the base branch from master to cmanallen/scm-platform February 2, 2026 21:11
GabeVillalobos
GabeVillalobos reviewed Feb 2, 2026
View reviewed changes
src/sentry/integrations/bitbucket/webhook.py
"organization_id": organization_id,
}
],
"type": "bitbucket",
Copy link
Member

@GabeVillalobos GabeVillalobos Feb 2, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If this helps, we have all of these provider names available in the IntegrationProviderSlug enum:

sentry/src/sentry/integrations/types.py

Lines 32 to 46 in 6469ade

class IntegrationProviderSlug(StrEnum):
SLACK = "slack"
DISCORD = "discord"
MSTEAMS = "msteams"
JIRA = "jira"
JIRA_SERVER = "jira_server"
AZURE_DEVOPS = "vsts"
GITHUB = "github"
GITHUB_ENTERPRISE = "github_enterprise"
GITLAB = "gitlab"
BITBUCKET = "bitbucket"
BITBUCKET_SERVER = "bitbucket_server"
PAGERDUTY = "pagerduty"
OPSGENIE = "opsgenie"
PERFORCE = "perforce"

Copy link
Member Author

@cmanallen cmanallen Feb 3, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Feb 2, 2026
alexsohn1126
alexsohn1126 reviewed Feb 3, 2026
View reviewed changes
src/sentry/scm/types.py
investigating the target integrations webhook.py file.
"""

sentry_meta: list["SubscriptionEventSentryMeta"] | None
Copy link
Member

@alexsohn1126 alexsohn1126 Feb 3, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is there a reason why we store the TypedDict instead of the actual objects themselves?

Also, when do we need multiple SubscriptionEventSentryMeta?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@GabeVillalobos GabeVillalobos GabeVillalobos left review comments

@alexsohn1126 alexsohn1126 alexsohn1126 left review comments

Assignees

No one assigned

Labels

Scope: Backend Automatically applied to PRs that change backend components

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@cmanallen @GabeVillalobos @alexsohn1126