Refactor webhook validation logic in project and user notifications to remove endpoint checks when isEnabled is true, streamlining the validation process.

Author: Dobrunia

src/resolvers/projectNotifications.ts

@@ -127,7 +127,7 @@ async function validateNotificationsRuleChannels(channels: NotificationsChannels
     }
   }
 
-  if (channels.webhook?.isEnabled && channels.webhook.endpoint) {
+  if (channels.webhook?.isEnabled) {
     const webhookError = await validateWebhookEndpoint(channels.webhook.endpoint);
 
     if (webhookError !== null) {

src/resolvers/userNotifications.ts

@@ -47,7 +47,7 @@ export default {
       { input }: ChangeUserNotificationsChannelPayload,
       { user, factories }: ResolverContextWithUser
     ): Promise<ChangeNotificationsResponse> {
-      if (input.webhook?.isEnabled && input.webhook.endpoint) {
+      if (input.webhook?.isEnabled) {
         const webhookError = await validateWebhookEndpoint(input.webhook.endpoint);
 
         if (webhookError !== null) {