Skip to content

fix(#19338): handle missing submitter in role verification flows#19340

Open
JacobCoffee wants to merge 11 commits intosentry-WAREHOUSE-PRODUCTION-2D6from
fix-submitter-none-handling
Open

fix(#19338): handle missing submitter in role verification flows#19340
JacobCoffee wants to merge 11 commits intosentry-WAREHOUSE-PRODUCTION-2D6from
fix-submitter-none-handling

Conversation

@JacobCoffee
Copy link
Member

Fix issues surfaced from fixing WAREHOUSE-PRODUCTION-2D6 that would throw errors on data access when a user is deleted

@JacobCoffee JacobCoffee requested a review from a team as a code owner January 16, 2026 21:37
@JacobCoffee JacobCoffee changed the title fix(#19338): none value for submitter fix(#19338): handle missing submitter in role verification flows Jan 16, 2026
@JacobCoffee JacobCoffee changed the base branch from main to sentry-WAREHOUSE-PRODUCTION-2D6 January 22, 2026 20:44
Copy link
Member

@ewdurbin ewdurbin left a comment

Choose a reason for hiding this comment

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

Alternative, rather than reaching this point, we could treat "The inviting user exists" as a condition for an invitation to be acceptable.

After line 1182, we could add a check for "inviting user still exists" and return _error(request._("Invalid Invitation: The inviting user no longer exists. Please ask a remaining Owner to reissue your invitation")) or similar...

@JacobCoffee JacobCoffee requested a review from ewdurbin February 19, 2026 19:05
@JacobCoffee JacobCoffee requested a review from a team February 19, 2026 20:27
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants