Skip to content

feat(gitlab): add inbound/outbound assignment & comment support#105472

Merged
GabeVillalobos merged 2 commits intomasterfrom
raj/gitlab-feat-par/assgn-comments
Jan 29, 2026
Merged

feat(gitlab): add inbound/outbound assignment & comment support#105472
GabeVillalobos merged 2 commits intomasterfrom
raj/gitlab-feat-par/assgn-comments

Conversation

@iamrajjoshi
Copy link
Collaborator

Take 2 because of #103877

This PR adds inbound/outbound assignment and comment synchronization support between Sentry and GitLab issues.

It is gated behind organizations:integrations-gitlab-project-management flag.

Comment Syncing

gitlab-comments.mov

Outbound Assignment Sync

gitlab-outbound.mov

Inbound Assignment Sync

gitlab-inbound.mov

Legal Boilerplate

Look, I get it. The entity doing business as "Sentry" was incorporated in the State of Delaware in 2015 as Functional Software, Inc. and is gonna need some rights from me in order to utilize my contributions in this here PR. So here's the deal: I retain all rights, title and interest in and to my contributions, and by keeping this boilerplate intact I confirm that Sentry can use, modify, copy, and redistribute my contributions, under Sentry's choice of terms.

@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Dec 27, 2025
@iamrajjoshi iamrajjoshi marked this pull request as ready for review December 27, 2025 02:41
@iamrajjoshi iamrajjoshi requested review from a team as code owners December 27, 2025 02:41
@iamrajjoshi iamrajjoshi force-pushed the raj/gitlab-feat-par/assgn-comments branch from 6f88618 to c3e315e Compare December 27, 2025 03:05
Comment on lines +92 to +98
provider = EXTERNAL_PROVIDERS_REVERSE[ExternalProviderEnum(self.model.provider)].value
external_actor = ExternalActor.objects.filter(
provider=provider,
user_id=user.id,
integration_id=external_issue.integration_id,
organization=external_issue.organization,
).first()
Copy link
Member

Choose a reason for hiding this comment

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

This logic seems so common between our providers, that I'm wondering if we have a way to abstract this out further. Just a thought.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

good call - i didn't do it originally since i hadn't run into multiple examples till now. we can def have separate utils for this which can be a followup.

@iamrajjoshi iamrajjoshi force-pushed the raj/gitlab-feat-par/assgn-comments branch from 2641751 to 06b22e3 Compare January 2, 2026 00:44
@iamrajjoshi iamrajjoshi force-pushed the raj/gitlab-feat-par/assgn-comments branch from 06b22e3 to b1f6462 Compare January 9, 2026 02:37
@iamrajjoshi iamrajjoshi added the Trigger: getsentry tests Once code is reviewed: apply label to PR to trigger getsentry tests label Jan 11, 2026
Copy link
Contributor

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 4 potential issues.

Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.

@GabeVillalobos GabeVillalobos merged commit 6cc1fb3 into master Jan 29, 2026
66 checks passed
@GabeVillalobos GabeVillalobos deleted the raj/gitlab-feat-par/assgn-comments branch January 29, 2026 22:26
@github-actions github-actions bot locked and limited conversation to collaborators Feb 14, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Scope: Backend Automatically applied to PRs that change backend components Trigger: getsentry tests Once code is reviewed: apply label to PR to trigger getsentry tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants