Skip to content

ref(settings): remove service hooks forms and routes #116296

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

Merged
TkDodo merged 5 commits into from
May 28, 2026
Merged

ref(settings): remove service hooks forms and routes #116296

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
b343755
ref(settings): Replace legacy FieldGroup with Scraps layout primitive…
TkDodo May 27, 2026
b0c6dd9
ref: use gap over padding
TkDodo May 27, 2026
247fd4e
Merge branch 'master' into tkdodo/migrate-service-hooks-fieldgroup
TkDodo May 27, 2026
9d4a611
ref(settings): remove service hooks forms and routes
TkDodo May 27, 2026
75e4634
fix: remove acceptance tests for deleted forms
TkDodo May 27, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
23 changes: 0 additions & 23 deletions static/app/router/routes.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -760,25 +760,6 @@ function buildRoutes(): RouteObject[] {
path: 'issue-tracking/',
redirectTo: '/settings/:orgId/:projectId/plugins/',
},
{
path: 'hooks/',
name: t('Service Hooks'),
component: make(() => import('sentry/views/settings/project/projectServiceHooks')),
},
{
path: 'hooks/new/',
name: t('Create Service Hook'),
component: make(
() => import('sentry/views/settings/project/projectCreateServiceHook')
),
},
{
path: 'hooks/:hookId/',
name: t('Service Hook Details'),
component: make(
() => import('sentry/views/settings/project/projectServiceHookDetails')
),
},
];
const projectSettingsRoutes: SentryRouteObject = {
path: 'projects/:projectId/',
Expand Down Expand Up @@ -2885,10 +2866,6 @@ function buildRoutes(): RouteObject[] {
path: 'filters/',
redirectTo: '/settings/:orgId/projects/:projectId/filters/',
},
{
path: 'hooks/',
redirectTo: '/settings/:orgId/projects/:projectId/hooks/',
},
{
path: 'keys/',
redirectTo: '/settings/:orgId/projects/:projectId/keys/',
Expand Down
12 changes: 0 additions & 12 deletions static/app/types/integrations.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -553,20 +553,8 @@ export type AppOrProviderOrPlugin =
| PluginWithProjectList
| DocIntegration;

/**
* Webhooks and servicehooks
*/
export type WebhookEvent = 'issue' | 'error' | 'comment' | 'seer' | 'preprod_artifact';

export type ServiceHook = {
dateCreated: string;
events: string[];
id: string;
secret: string;
status: string;
url: string;
};

/**
* Codeowners and repository path mappings.
*/
Expand Down
27 changes: 0 additions & 27 deletions static/app/views/settings/project/projectCreateServiceHook.tsx
View file
Open in desktop

This file was deleted.

225 changes: 0 additions & 225 deletions static/app/views/settings/project/projectServiceHookDetails.tsx
View file
Open in desktop

This file was deleted.

Loading
Loading