Merge pull request #34 from AmRo045/dev

Fix disabling notification bug

Author: AmRo045

package.json

@@ -1,6 +1,6 @@
 {
   "name": "outline-admin",
-  "version": "0.3.4",
+  "version": "0.3.5",
   "private": true,
   "scripts": {
     "compile": "tsc -p tsconfig.scripts.json && tscpaths -p tsconfig.scripts.tscpaths.json -s ./dist -o ./dist",

src/components/health-check-edit-form.tsx

@@ -48,7 +48,7 @@ export default function HealthCheckEditForm({ healthCheck, notificationChannels
         try {
             await updateHealthCheck({
                 id: healthCheck.id,
-                notificationChannelId: channelId > 0 ? channelId : undefined,
+                notificationChannelId: channelId > 0 ? channelId : null,
                 notificationCooldown: data.notificationCooldown,
                 interval: data.interval
             });

src/core/definitions.ts

@@ -151,7 +151,7 @@ export interface DynamicAccessKeyApiResponse {
 
 export interface UpdateHealthCheckRequest {
     id: number;
-    notificationChannelId?: number;
+    notificationChannelId?: number | null;
     notificationCooldown: number;
     interval: number;
 }