Skip to content

ref(flags): Remove organizations:new-organization-member-invite#114812

Open
wedamija wants to merge 2 commits intomasterfrom
flag-cleanup/organizations-new-organization-member-invite
Open

ref(flags): Remove organizations:new-organization-member-invite#114812
wedamija wants to merge 2 commits intomasterfrom
flag-cleanup/organizations-new-organization-member-invite

Conversation

@wedamija
Copy link
Copy Markdown
Member

@wedamija wedamija commented May 4, 2026

The flag scanner classified this as flagpole-disabled. The flag gated the entire invited-members API surface — index, details, and reinvite endpoints all returned 403 MISSING_FEATURE when False. Removes the never-reachable endpoints (but keeps the OrganizationMemberInvite model and its serializers), their URL routes, and the corresponding tests.

The flag scanner classified this as flagpole-disabled. The flag gated
the entire invited-members API surface — index, details, and reinvite
endpoints all returned 403 MISSING_FEATURE when False. Removes the
never-reachable endpoints (but keeps the OrganizationMemberInvite
model and its serializers), their URL routes, and the corresponding
tests.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@wedamija wedamija requested a review from a team as a code owner May 4, 2026 23:18
@github-actions github-actions Bot added the Scope: Backend Automatically applied to PRs that change backend components label May 4, 2026
@getsantry getsantry Bot requested a review from a team as a code owner May 4, 2026 23:20
@github-actions github-actions Bot added the Scope: Frontend Automatically applied to PRs that change frontend components label May 4, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 4, 2026

🚨 Warning: This pull request contains Frontend and Backend changes!

It's discouraged to make changes to Sentry's Frontend and Backend in a single pull request. The Frontend and Backend are not atomically deployed. If the changes are interdependent of each other, they must be separated into two pull requests and be made forward or backwards compatible, such that the Backend or Frontend can be safely deployed independently.

Have questions? Please ask in the #discuss-dev-infra channel.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 4, 2026

📊 Type Coverage Diff

✅ No new type safety issues introduced. Coverage: 93.40%

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 4, 2026

Backend Test Failures

Failures on 9fb7a94 in this run:

tests/sentry/test_mypy_stronglist.py::test_stronglistlog
[gw0] linux -- Python 3.13.1 /home/runner/work/sentry/sentry/.venv/bin/python3
tests/sentry/test_mypy_stronglist.py:20: in test_stronglist
    raise AssertionError(f"\n\n{proc.stdout}")
E   AssertionError: 
E   
E   pyproject.toml: tests.sentry.api.endpoints.test_organization_member_reinvite in weaklist does not match any files!

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

Labels

Scope: Backend Automatically applied to PRs that change backend components Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant