Handle side effects when logged in user is added as a collaborator

mathesar-foundation · Oct 1, 2024 · 98e48ea · 98e48ea
1 parent 4291b3b
commit 98e48ea
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 1 deletion.
diff --git a/mathesar_ui/src/contexts/DatabaseSettingsRouteContext.ts b/mathesar_ui/src/contexts/DatabaseSettingsRouteContext.ts
@@ -137,6 +137,7 @@ export class DatabaseSettingsRouteContext {
     this.collaborators.updateResolvedValue((collaborators) =>
       collaborators.with(newCollaborator.id, newCollaborator),
     );
+    return newCollaborator;
   }
 
   async deleteCollaborator(collaborator: Collaborator) {

diff --git a/mathesar_ui/src/pages/database/settings/collaborators/AddCollaboratorModal.svelte b/mathesar_ui/src/pages/database/settings/collaborators/AddCollaboratorModal.svelte
@@ -28,6 +28,7 @@
   export let configuredRolesMap: ImmutableMap<number, ConfiguredRole>;
   export let usersMap: ImmutableMap<number, User>;
   export let collaboratorsMap: ImmutableMap<number, Collaborator>;
+  export let onAdd: (collaborator: Collaborator) => void;
 
   const userId = requiredField<number | undefined>(undefined);
   const configuredRoleId = requiredField<number | undefined>(undefined);
@@ -41,7 +42,11 @@
 
   async function addCollaborator() {
     if ($userId && $configuredRoleId) {
-      await $routeContext.addCollaborator($userId, $configuredRoleId);
+      const collaborator = await $routeContext.addCollaborator(
+        $userId,
+        $configuredRoleId,
+      );
+      onAdd(collaborator);
       controller.close();
       toast.success($_('collaborator_added_successfully'));
       form.reset();

diff --git a/mathesar_ui/src/pages/database/settings/collaborators/Collaborators.svelte b/mathesar_ui/src/pages/database/settings/collaborators/Collaborators.svelte
@@ -117,6 +117,7 @@
     usersMap={$users.resolvedValue}
     configuredRolesMap={$configuredRoles.resolvedValue}
     collaboratorsMap={$collaborators.resolvedValue}
+    onAdd={checkAndHandleSideEffects}
   />
 {/if}