From a1553ac0bfef8bc137934ac604ac880834e4ed08 Mon Sep 17 00:00:00 2001
From: baldo <baldo@toppoint.de>
Date: Thu, 25 Aug 2022 21:10:37 +0200
Subject: [PATCH 1/3] Replace EnumValue type by more general TypeOf type.

---
 server/shared/types/index.ts | 15 +++++----------
 server/utils/resources.ts    |  8 ++++----
 2 files changed, 9 insertions(+), 14 deletions(-)

diff --git a/server/shared/types/index.ts b/server/shared/types/index.ts
index 5c8a77e..b4677d8 100644
--- a/server/shared/types/index.ts
+++ b/server/shared/types/index.ts
@@ -1,9 +1,4 @@
-import {
-    ArrayField,
-    Field,
-    RawJsonField,
-    registerStringMapper,
-} from "sparkson";
+import { ArrayField, Field, RawJsonField } from "sparkson";
 
 // Types shared with the client.
 export type TypeGuard<T> = (arg: unknown) => arg is T;
@@ -71,8 +66,8 @@ export type JSONArray = Array<JSONValue>;
 
 export const isJSONArray = toIsArray(isJSONValue);
 
-export type EnumValue<E> = E[keyof E];
-export type EnumTypeGuard<E> = TypeGuard<EnumValue<E>>;
+export type ValueOf<T> = T[keyof T];
+export type EnumTypeGuard<E> = TypeGuard<ValueOf<E>>;
 
 export function unhandledEnumField(field: never): never {
     throw new Error(`Unhandled enum field: ${field}`);
@@ -138,7 +133,7 @@ export function isBoolean(arg: unknown): arg is boolean {
 }
 
 export function isUndefined(arg: unknown): arg is undefined {
-    return arg === undefined;
+    return typeof arg === "undefined";
 }
 
 export function isNull(arg: unknown): arg is null {
@@ -150,7 +145,7 @@ export function toIsArray<T>(isT: TypeGuard<T>): TypeGuard<T[]> {
 }
 
 export function toIsEnum<E>(enumDef: E): EnumTypeGuard<E> {
-    return (arg): arg is EnumValue<E> =>
+    return (arg): arg is ValueOf<E> =>
         Object.values(enumDef).includes(arg as [keyof E]);
 }
 
diff --git a/server/utils/resources.ts b/server/utils/resources.ts
index 218c939..e218f1a 100644
--- a/server/utils/resources.ts
+++ b/server/utils/resources.ts
@@ -12,7 +12,7 @@ import {
 import { Request, Response } from "express";
 import {
     EnumTypeGuard,
-    EnumValue,
+    ValueOf,
     type GenericSortField,
     getFieldIfExists,
     isJSONObject,
@@ -79,10 +79,10 @@ function respond(
 
 function orderByClause<S>(
     restParams: RestParams,
-    defaultSortField: EnumValue<S>,
+    defaultSortField: ValueOf<S>,
     isSortField: EnumTypeGuard<S>
 ): OrderByClause {
-    let sortField: EnumValue<S> | undefined = isSortField(restParams._sortField)
+    let sortField: ValueOf<S> | undefined = isSortField(restParams._sortField)
         ? restParams._sortField
         : undefined;
     if (!sortField) {
@@ -347,7 +347,7 @@ export { filterCondition as whereCondition };
 
 export function filterClause<S>(
     restParams: RestParams,
-    defaultSortField: EnumValue<S>,
+    defaultSortField: ValueOf<S>,
     isSortField: EnumTypeGuard<S>,
     filterFields: string[]
 ): FilterClause {

From 0e30ec1f964eee347e9728c2b96e7b2f1e6ba47e Mon Sep 17 00:00:00 2001
From: baldo <baldo@toppoint.de>
Date: Thu, 25 Aug 2022 22:14:59 +0200
Subject: [PATCH 2/3] Get rid of any.

---
 server/shared/types/index.ts | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/server/shared/types/index.ts b/server/shared/types/index.ts
index b4677d8..016fee4 100644
--- a/server/shared/types/index.ts
+++ b/server/shared/types/index.ts
@@ -695,7 +695,13 @@ export type NodesFilter = {
     onlineState?: OnlineState;
 };
 
-export const NODES_FILTER_FIELDS = {
+export const NODES_FILTER_FIELDS: Record<
+    keyof NodesFilter,
+    | BooleanConstructor
+    | StringConstructor
+    | typeof MonitoringState
+    | typeof OnlineState
+> = {
     hasKey: Boolean,
     hasCoords: Boolean,
     monitoringState: MonitoringState,

From fd853c7319c023d5c4a883bd7066ef3b0afe6d3c Mon Sep 17 00:00:00 2001
From: baldo <baldo@toppoint.de>
Date: Wed, 24 Aug 2022 23:48:03 +0200
Subject: [PATCH 3/3] Show node on map in confirmation before deleting it.

---
 server/shared/types/index.ts | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/server/shared/types/index.ts b/server/shared/types/index.ts
index 016fee4..5fb3c06 100644
--- a/server/shared/types/index.ts
+++ b/server/shared/types/index.ts
@@ -461,6 +461,10 @@ export const isHostname = toIsNewtype(isString, "" as Hostname);
 export type Nickname = string & { readonly __tag: unique symbol };
 export const isNickname = toIsNewtype(isString, "" as Nickname);
 
+/**
+ * String representing geo coordinates. Latitude and longitude are delimited by one whitespace.
+ * E.g.: <code>"53.565278 10.001389"</code>
+ */
 export type Coordinates = string & { readonly __tag: unique symbol };
 export const isCoordinates = toIsNewtype(isString, "" as Coordinates);