Skip to content

ref(integrations): Remove unused legacy extension configuration view#117338

Merged
evanpurkhiser merged 1 commit into
masterfrom
evanpurkhiser/ref-integrations-remove-unused-legacy-extension-configuration-view
Jun 10, 2026
Merged

ref(integrations): Remove unused legacy extension configuration view#117338
evanpurkhiser merged 1 commit into
masterfrom
evanpurkhiser/ref-integrations-remove-unused-legacy-extension-configuration-view

Conversation

@evanpurkhiser

@evanpurkhiser evanpurkhiser commented Jun 10, 2026

Copy link
Copy Markdown
Member

IntegrationExtensionConfigurationView (and its ExternalIntegrationPipeline helper) backed the legacy server-rendered /extensions/<slug>/configure/ install flow. Every provider now installs through the API-driven pipeline, the configure URL no longer routes to this view, and nothing in sentry or getsentry references it — so it's dead code.

Part of retiring the legacy install path now that all integration providers are API-driven. The broader legacy web-pipeline machinery is intentionally left in place for now (still shared with auth/SSO, and being retired provider-by-provider).

Part of VDY-32.

`IntegrationExtensionConfigurationView` (and its `ExternalIntegrationPipeline`
helper) backed the legacy server-rendered `/extensions/<slug>/configure/`
install flow. Every provider now installs through the API-driven pipeline,
the configure URL no longer routes to this view, and nothing in sentry or
getsentry references it.
@evanpurkhiser evanpurkhiser requested a review from markstory June 10, 2026 18:02
@github-actions github-actions Bot added the Scope: Backend Automatically applied to PRs that change backend components label Jun 10, 2026
@evanpurkhiser evanpurkhiser requested review from a team and wedamija June 10, 2026 18:24
@linear-code

linear-code Bot commented Jun 10, 2026

Copy link
Copy Markdown

VDY-32

@evanpurkhiser evanpurkhiser enabled auto-merge (squash) June 10, 2026 20:19

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

It was a base view, so there were no URLs pointing at it

@wedamija wedamija left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

🔥

@evanpurkhiser evanpurkhiser merged commit 49523a4 into master Jun 10, 2026
66 checks passed
@evanpurkhiser evanpurkhiser deleted the evanpurkhiser/ref-integrations-remove-unused-legacy-extension-configuration-view branch June 10, 2026 21:41
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants