From 85af2c087520ccc00c62bffec1575707678298aa Mon Sep 17 00:00:00 2001 From: Pujit Mehrotra Date: Thu, 28 Aug 2025 16:05:55 -0400 Subject: [PATCH 1/2] fix: confusing `server -> status` query represent the target server's status instead of whether it's connected to Mothership. Resolves #1627 --- .../resolvers/servers/server.resolver.ts | 35 ++++++++----------- 1 file changed, 15 insertions(+), 20 deletions(-) diff --git a/api/src/unraid-api/graph/resolvers/servers/server.resolver.ts b/api/src/unraid-api/graph/resolvers/servers/server.resolver.ts index f5e3e87795..8bcc2e9e3f 100644 --- a/api/src/unraid-api/graph/resolvers/servers/server.resolver.ts +++ b/api/src/unraid-api/graph/resolvers/servers/server.resolver.ts @@ -24,7 +24,7 @@ export class ServerResolver { resource: Resource.SERVERS, }) public async server(): Promise { - return this.getLocalServer()[0] || null; + return this.getLocalServer() || null; } @Query(() => [ServerModel]) @@ -33,7 +33,7 @@ export class ServerResolver { resource: Resource.SERVERS, }) public async servers(): Promise { - return this.getLocalServer(); + return [this.getLocalServer()]; } @Subscription(() => ServerModel) @@ -45,7 +45,7 @@ export class ServerResolver { return createSubscription(PUBSUB_CHANNEL.SERVERS); } - private getLocalServer(): ServerModel[] { + private getLocalServer(): ServerModel { const emhttp = getters.emhttp(); const connectConfig = this.configService.get('connect'); @@ -64,22 +64,17 @@ export class ServerResolver { avatar: '', }; - return [ - { - id: 'local', - owner, - guid: guid || '', - apikey: connectConfig?.config?.apikey ?? '', - name: name ?? 'Local Server', - status: - connectConfig?.mothership?.status === MinigraphStatus.CONNECTED - ? ServerStatus.ONLINE - : ServerStatus.OFFLINE, - wanip, - lanip, - localurl, - remoteurl, - }, - ]; + return { + id: 'local', + owner, + guid: guid || '', + apikey: connectConfig?.config?.apikey ?? '', + name: name ?? 'Local Server', + status: ServerStatus.ONLINE, + wanip, + lanip, + localurl, + remoteurl, + }; } } From 7fc4992d9866e7c8cf13a60b596a32097d86fda4 Mon Sep 17 00:00:00 2001 From: Pujit Mehrotra Date: Thu, 28 Aug 2025 16:09:20 -0400 Subject: [PATCH 2/2] add description to server status field --- api/src/unraid-api/graph/resolvers/servers/server.model.ts | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/api/src/unraid-api/graph/resolvers/servers/server.model.ts b/api/src/unraid-api/graph/resolvers/servers/server.model.ts index d17ec47bac..dc7b94c95c 100644 --- a/api/src/unraid-api/graph/resolvers/servers/server.model.ts +++ b/api/src/unraid-api/graph/resolvers/servers/server.model.ts @@ -38,7 +38,9 @@ export class Server extends Node { @Field() name!: string; - @Field(() => ServerStatus) + @Field(() => ServerStatus, { + description: 'Whether this server is online or offline', + }) status!: ServerStatus; @Field()