feat: site transfer mutation (#148)

* feat: Add service method to transfer sites

* feat: Add reducer for site transfer mutation

* feat: Include unaffiliated sites

package.json

@@ -11,7 +11,7 @@
     "lodash": "^4.17.21",
     "react": "^18.2.0",
     "react-redux": "^8.1.3",
-    "terraso-backend": "github:techmatters/terraso-backend#3d323b5",
+    "terraso-backend": "github:techmatters/terraso-backend#f671497",
     "uuid": "^9.0.1"
   },
   "scripts": {

src/selectors.test.ts

@@ -55,10 +55,10 @@ const generateProject = (
   };
 };
 
-const generateSite = (project: Project): Site => {
+const generateSite = (project?: Project): Site => {
   const id = uuidv4();
   const site: Site = {
-    projectId: project.id,
+    projectId: project?.id,
     ownerId: undefined,
     id,
     name: 'Test Site',
@@ -68,7 +68,9 @@ const generateSite = (project: Project): Site => {
     archived: false,
     updatedAt: '2023-10-24',
   };
-  project.sites[site.id] = true;
+  if (project !== undefined) {
+    project.sites[site.id] = true;
+  }
   return site;
 };
 
@@ -157,9 +159,10 @@ test('can access all projects with role', () => {
   const project3 = generateProject([generateMembership(user.id, 'manager')]);
   const site1 = generateSite(project1);
   const site2 = generateSite(project2);
+  const site3 = generateSite();
 
   const store = createStore(
-    initState([project1, project2, project3], [user], [site1, site2]),
+    initState([project1, project2, project3], [user], [site1, site2, site3]),
   );
   const pairs = selectProjectsWithTransferrableSites(
     store.getState(),
@@ -178,5 +181,6 @@ test('can access all projects with role', () => {
         siteName: site1.name,
       },
     ],
+    unaffiliatedSites: [{ siteId: site3.id, siteName: site3.name }],
   });
 });

src/selectors.ts

@@ -52,13 +52,17 @@ export const selectProjectsWithTransferrableSites = createSelector(
           };
         }),
     );
-    let projectRecord = projects.reduce(
+
+    const unaffiliatedSites = Object.values(sites)
+      .filter(({ projectId }) => projectId === undefined)
+      .map(({ id, name }) => ({ siteId: id, siteName: name }));
+    const projectRecord = projects.reduce(
       (x, { id, name }) => ({
         ...x,
         [id]: { projectId: id, projectName: name },
       }),
       {} as Record<string, { projectId: string; projectName: string }>,
     );
-    return { projects: projectRecord, sites: projectSites };
+    return { projects: projectRecord, sites: projectSites, unaffiliatedSites };
   },
 );

src/site/siteService.ts

@@ -20,6 +20,7 @@ import { graphql } from 'terraso-client-shared/graphqlSchema';
 import type {
   SiteAddMutationInput,
   SiteDataFragment,
+  SiteTransferMutationInput,
   SiteUpdateMutationInput,
 } from 'terraso-client-shared/graphqlSchema/graphql';
 import type { Site } from 'terraso-client-shared/site/siteSlice';
@@ -154,3 +155,42 @@ export const deleteSite = (site: Site) => {
     .requestGraphQL(query, { input: { id: site.id } })
     .then(_ => site.id);
 };
+
+export const transferSitesToProject = (input: SiteTransferMutationInput) => {
+  const query = graphql(`
+    mutation transferSites($input: SiteTransferMutationInput!) {
+      transferSites(input: $input) {
+        project {
+          id
+        }
+        updated {
+          site {
+            id
+          }
+          oldProject {
+            id
+          }
+        }
+      }
+    }
+  `);
+  return terrasoApi.requestGraphQL(query, { input }).then(
+    ({
+      transferSites: {
+        project: { id: projectId },
+        updated,
+      },
+    }) => {
+      const condensedUpdated = updated.map(
+        ({ site: { id: siteId }, oldProject }) => {
+          const obj: { siteId: string; oldProjectId?: string } = { siteId };
+          if (oldProject) {
+            obj.oldProjectId = oldProject.id;
+          }
+          return obj;
+        },
+      );
+      return { projectId, updated: condensedUpdated };
+    },
+  );
+};

src/site/siteSlice.ts

@@ -16,10 +16,14 @@
  */
 
 import { createSlice, PayloadAction } from '@reduxjs/toolkit';
-import { SiteAddMutationInput } from 'terraso-client-shared/graphqlSchema/graphql';
+import {
+  SiteAddMutationInput,
+  SiteTransferMutationInput,
+} from 'terraso-client-shared/graphqlSchema/graphql';
 import {
   addSiteToProject,
   removeSiteFromAllProjects,
+  removeSiteFromProject,
 } from 'terraso-client-shared/project/projectSlice';
 import * as siteService from 'terraso-client-shared/site/siteService';
 import { createAsyncThunk } from 'terraso-client-shared/store/utils';
@@ -82,6 +86,20 @@ export const deleteSite = createAsyncThunk<string, Site>(
   },
 );
 
+export const transferSites = createAsyncThunk<
+  Awaited<ReturnType<typeof siteService.transferSitesToProject>>,
+  SiteTransferMutationInput
+>('site/transferSites', async (input, _currentUser, { dispatch }) => {
+  const result = await siteService.transferSitesToProject(input);
+  for (const { siteId, oldProjectId } of result.updated) {
+    if (oldProjectId !== undefined) {
+      dispatch(removeSiteFromProject({ siteId, projectId: oldProjectId }));
+    }
+    dispatch(addSiteToProject({ siteId, projectId: result.projectId }));
+  }
+  return result;
+});
+
 const siteSlice = createSlice({
   name: 'site',
   initialState,
@@ -133,6 +151,15 @@ const siteSlice = createSlice({
     builder.addCase(deleteSite.fulfilled, (state, { meta }) => {
       delete state.sites[meta.arg.id];
     });
+
+    builder.addCase(
+      transferSites.fulfilled,
+      (state, { payload: { projectId, updated } }) => {
+        for (const { siteId } of updated) {
+          state.sites[siteId].projectId = projectId;
+        }
+      },
+    );
   },
 });