Skip to content

Commit 0a5a5f8

Browse files
mtopo27claude
andcommitted
fix(notifications): Handle null provider in organization integrations filter
The organization integrations filter crashes when an entry has a null provider, causing the notification settings page to error. Add optional chaining to match the pattern already used for identity filtering. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 parent 4dda701 commit 0a5a5f8

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

static/app/views/settings/account/notifications/notificationSettingsByType.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -103,7 +103,7 @@ export function NotificationSettingsByType({notificationType}: Props) {
103103
{staleTime: 30_000}
104104
);
105105
const organizationIntegrations = allOrgIntegrations.filter(orgIntegration =>
106-
ALLOWED_PROVIDERS.has(orgIntegration.provider.key as SupportedProviders)
106+
ALLOWED_PROVIDERS.has(orgIntegration?.provider?.key as SupportedProviders)
107107
);
108108
const {data: defaultSettings, status: defaultSettingsStatus} =
109109
useApiQuery<DefaultSettings>([getApiUrl('/notification-defaults/')], {

0 commit comments

Comments
 (0)