From 4217eedad542ace3628d330b8fb791aa8322bc00 Mon Sep 17 00:00:00 2001 From: Hakan Shehu Date: Fri, 30 Jan 2026 17:12:02 +0100 Subject: [PATCH] Fix workspace update (#310) --- .../mutations/workspaces/workspace-update.ts | 17 +++-------------- .../mutations/workspaces/workspace-update.ts | 3 +-- .../workspaces/workspace-settings-container.tsx | 7 ++----- 3 files changed, 6 insertions(+), 21 deletions(-) diff --git a/packages/client/src/handlers/mutations/workspaces/workspace-update.ts b/packages/client/src/handlers/mutations/workspaces/workspace-update.ts index bbb0b4f7..0e3786da 100644 --- a/packages/client/src/handlers/mutations/workspaces/workspace-update.ts +++ b/packages/client/src/handlers/mutations/workspaces/workspace-update.ts @@ -12,9 +12,7 @@ import { AppService } from '@colanode/client/services/app-service'; import { Workspace } from '@colanode/client/types'; import { WorkspaceUpdateInput } from '@colanode/core'; -export class WorkspaceUpdateMutationHandler - implements MutationHandler -{ +export class WorkspaceUpdateMutationHandler implements MutationHandler { private readonly app: AppService; constructor(app: AppService) { @@ -24,15 +22,6 @@ export class WorkspaceUpdateMutationHandler async handleMutation( input: WorkspaceUpdateMutationInput ): Promise { - const accountService = this.app.getAccount(input.userId); - - if (!accountService) { - throw new MutationError( - MutationErrorCode.AccountNotFound, - 'Account not found or has been logged out.' - ); - } - const workspaceService = this.app.getWorkspace(input.userId); if (!workspaceService) { throw new MutationError( @@ -48,8 +37,8 @@ export class WorkspaceUpdateMutationHandler avatar: input.avatar, }; - const response = await accountService.client - .patch(`v1/workspaces/${input.id}`, { + const response = await workspaceService.account.client + .patch(`v1/workspaces/${workspaceService.workspace.workspaceId}`, { json: body, }) .json(); diff --git a/packages/client/src/mutations/workspaces/workspace-update.ts b/packages/client/src/mutations/workspaces/workspace-update.ts index 2080a505..04f556d2 100644 --- a/packages/client/src/mutations/workspaces/workspace-update.ts +++ b/packages/client/src/mutations/workspaces/workspace-update.ts @@ -1,10 +1,9 @@ export type WorkspaceUpdateMutationInput = { type: 'workspace.update'; - id: string; + userId: string; name: string; description: string; avatar: string | null; - userId: string; }; export type WorkspaceUpdateMutationOutput = { diff --git a/packages/ui/src/components/workspaces/workspace-settings-container.tsx b/packages/ui/src/components/workspaces/workspace-settings-container.tsx index 0c53191d..8fe02c69 100644 --- a/packages/ui/src/components/workspaces/workspace-settings-container.tsx +++ b/packages/ui/src/components/workspaces/workspace-settings-container.tsx @@ -20,15 +20,13 @@ export const WorkspaceSettingsContainer = () => { (q) => q .from({ workspaces: collections.workspaces }) - .where(({ workspaces }) => - eq(workspaces.accountId, workspace.accountId) - ) + .where(({ workspaces }) => eq(workspaces.userId, workspace.userId)) .select(({ workspaces }) => ({ name: workspaces.name, description: workspaces.description, avatar: workspaces.avatar, })), - [workspace.accountId] + [workspace.userId] ); const currentWorkspace = currentWorkspaceQuery.data?.[0]; @@ -57,7 +55,6 @@ export const WorkspaceSettingsContainer = () => { mutate({ input: { type: 'workspace.update', - id: workspace.workspaceId, userId: workspace.userId, name: values.name, description: values.description,