Deploy

src/lib/featureFlags.ts

@@ -12,7 +12,6 @@ const FEATURE_FLAGS_LIST = [
   "diamond_blurhash-enabled-globally",
   "onyx_enable-home-view-section-featured-fairs",
   "diamond_home-view-marketing-collection-categories",
-  "emerald_home-view-tasks-section",
   "onyx_experiment_home_view_test",
 ] as const
 

src/lib/loaders/loaders_with_authentication/gravity.ts

@@ -256,6 +256,21 @@ export default (accessToken, userID, opts) => {
       {},
       { method: "PUT" }
     ),
+    updateViewingRoomLoader: gravityLoader(
+      (id) => `viewing_room/${id}`,
+      {},
+      { method: "PUT" }
+    ),
+    updateViewingRoomArtworksLoader: gravityLoader(
+      (id) => `viewing_room/${id}/viewing_room_artworks`,
+      {},
+      { method: "PUT" }
+    ),
+    updateViewingRoomSubsectionsLoader: gravityLoader(
+      (id) => `viewing_room/${id}/subsections`,
+      {},
+      { method: "PUT" }
+    ),
     deleteHeroUnitLoader: gravityLoader(
       (id) => `hero_units/${id}`,
       {},
@@ -289,6 +304,11 @@ export default (accessToken, userID, opts) => {
       {},
       { method: "DELETE" }
     ),
+    deleteViewingRoomLoader: gravityLoader(
+      (id) => `viewing_room/${id}`,
+      {},
+      { method: "DELETE" }
+    ),
     dislikeArtworkLoader: gravityLoader(
       (id) => `collection/disliked-artwork/artwork/${id}`,
       {},

src/lib/stitching/gravity/schema.ts

@@ -58,13 +58,30 @@ export const executableGravitySchema = () => {
 
     duplicatedTypes.push("CreateViewingRoomPayload")
     duplicatedTypes.push("CreateViewingRoomInput")
+    duplicatedTypes.push("DeleteViewingRoomInput")
+    duplicatedTypes.push("DeleteViewingRoomPayload")
     duplicatedTypes.push("ViewingRoomOrErrorsUnion")
     duplicatedTypes.push("ViewingRoomAttributes")
+    duplicatedTypes.push("UpdateViewingRoomPayload")
+    duplicatedTypes.push("UpdateViewingRoomArtworksInput")
+    duplicatedTypes.push("PublishViewingRoomInput")
+    duplicatedTypes.push("PublishViewingRoomPayload")
+    duplicatedTypes.push("UnpublishViewingRoomInput")
+    duplicatedTypes.push("UnpublishViewingRoomPayload")
+    duplicatedTypes.push("UpdateViewingRoomArtworksPayload")
+    duplicatedTypes.push("UpdateViewingRoomSubsectionsInput")
+    duplicatedTypes.push("UpdateViewingRoomSubsectionsPayload")
   }
 
   const excludedMutations: string[] = []
   if (config.USE_UNSTITCHED_VIEWING_ROOM_SCHEMA) {
     excludedMutations.push("createViewingRoom")
+    excludedMutations.push("deleteViewingRoom")
+    excludedMutations.push("publishViewingRoom")
+    excludedMutations.push("unpublishViewingRoom")
+    excludedMutations.push("updateViewingRoom")
+    excludedMutations.push("updateViewingRoomArtworks")
+    excludedMutations.push("updateViewingRoomSubsections")
   }
 
   // Types which come from Gravity that are not (yet) needed in MP.

src/schema/v2/artwork/__tests__/artwork.test.js

@@ -5129,6 +5129,7 @@ describe("Artwork type", () => {
       describe("runningShow", () => {
         it("returns the show or fair if the artwork id is in a running show or fair", async () => {
           artwork.purchasable = true
+          artwork.show_ids = ["show-id"]
           context.showsLoader.mockResolvedValue([
             {
               name: "Test Show",

src/schema/v2/artwork/collectorSignals.ts

@@ -185,9 +185,12 @@ export const CollectorSignals: GraphQLFieldConfig<any, ResolverContext> = {
         type: Show.type,
         description:
           "Most recent running Show or Fair booth the artwork is currently in, sorted by relevance",
-        resolve: async (artwork, {}, ctx) => {
+        resolve: async ({ show_ids }, {}, ctx) => {
+          if (!show_ids || show_ids.length === 0) {
+            return null
+          }
           const showOrFair = await await ctx.showsLoader({
-            artwork: artwork._id,
+            id: show_ids,
             size: 1,
             status: "running",
             has_location: true,

src/schema/v2/artwork/context.ts

@@ -34,7 +34,7 @@ const Context: GraphQLFieldConfig<any, ResolverContext> = {
   type: ArtworkContextType,
   description: "Returns the associated Fair/Sale/Show",
   resolve: (
-    { id, sale_ids },
+    { id, sale_ids, show_ids },
     _options,
     { salesLoader, relatedFairsLoader, showsLoader }
   ) => {
@@ -58,21 +58,24 @@ const Context: GraphQLFieldConfig<any, ResolverContext> = {
         return assign({ context_type: "Fair" }, fair)
       })
 
-    const show_promise = showsLoader({
-      artwork: id,
-      size: 1,
-      at_a_fair: false,
-    })
-      .then(first)
-      .then((show) => {
-        if (!show) return null
-        return assign({ context_type: "Show" }, show)
+    let showPromise
+    if (show_ids && show_ids.length > 0) {
+      showPromise = showsLoader({
+        id: show_ids,
+        size: 1,
+        at_a_fair: false,
       })
+        .then(first)
+        .then((show) => {
+          if (!show) return null
+          return assign({ context_type: "Show" }, show)
+        })
+    }
 
     return Promise.all([
       sale_promise || Promise.resolve(null),
       fair_promise,
-      show_promise,
+      showPromise || Promise.resolve(null),
     ]).then(choose)
   },
 }

src/schema/v2/homeView/__tests__/HomeView.test.ts

@@ -174,6 +174,14 @@ describe("homeView", () => {
         expect(homeView.sectionsConnection).toMatchInlineSnapshot(`
           {
             "edges": [
+              {
+                "node": {
+                  "__typename": "HomeViewSectionTasks",
+                  "component": {
+                    "title": "Act Now",
+                  },
+                },
+              },
               {
                 "node": {
                   "__typename": "HomeViewSectionActivity",

src/schema/v2/homeView/sections/Tasks.ts

@@ -13,7 +13,6 @@ export const Tasks: HomeViewSection = {
   component: {
     title: "Act Now",
   },
-  featureFlag: "emerald_home-view-tasks-section",
   requiresAuthentication: true,
   resolver: withHomeViewTimeout(async (_parent, args, { meTasksLoader }) => {
     if (!meTasksLoader) return null

src/schema/v2/homeView/sections/__tests__/Tasks.test.ts

@@ -1,20 +1,11 @@
-import { isFeatureFlagEnabled } from "lib/featureFlags"
 import gql from "lib/gql"
 import { runQuery } from "schema/v2/test/utils"
 
 jest.mock("lib/featureFlags", () => ({
   isFeatureFlagEnabled: jest.fn(() => true),
 }))
 
-const mockIsFeatureFlagEnabled = isFeatureFlagEnabled as jest.Mock
-
 describe("HomeViewSectionTasks", () => {
-  beforeAll(() => {
-    mockIsFeatureFlagEnabled.mockImplementation((flag: string) => {
-      if (flag === "emerald_home-view-tasks-section") return true
-    })
-  })
-
   it("returns the section's metadata", async () => {
     const query = gql`
       {
@@ -133,33 +124,4 @@ describe("HomeViewSectionTasks", () => {
       }
     `)
   })
-
-  describe("when the feature flag is disabled", () => {
-    beforeAll(() => {
-      mockIsFeatureFlagEnabled.mockImplementation((flag: string) => {
-        if (flag === "emerald_home-view-tasks-section") return false
-      })
-    })
-
-    it("throws an error when accessed by id", async () => {
-      const query = gql`
-        {
-          homeView {
-            section(id: "home-view-section-tasks") {
-              __typename
-              component {
-                title
-              }
-            }
-          }
-        }
-      `
-
-      const context = {}
-
-      await expect(runQuery(query, context)).rejects.toThrow(
-        "Section is not displayable: home-view-section-tasks"
-      )
-    })
-  })
 })

src/schema/v2/schema.ts

@@ -261,6 +261,12 @@ import { deliverSecondFactorMutation } from "./me/secondFactors/mutations/delive
 import { enableSecondFactorMutation } from "./me/secondFactors/mutations/enableSecondFactor"
 import { createAndSendBackupSecondFactorMutation } from "./users/createAndSendBackupSecondFactorMutation"
 import { createViewingRoomMutation } from "./viewingRooms/mutations/createViewingRoomMutation"
+import { updateViewingRoomMutation } from "./viewingRooms/mutations/updateViewingRoomMutation"
+import { deleteViewingRoomMutation } from "./viewingRooms/mutations/deleteViewingRoomMutation"
+import { publishViewingRoomMutation } from "./viewingRooms/mutations/publishViewingRoomMutation"
+import { unpublishViewingRoomMutation } from "./viewingRooms/mutations/unpublishViewingRoomMutation"
+import { updateViewingRoomArtworksMutation } from "./viewingRooms/mutations/updateViewingRoomArtworks"
+import { updateViewingRoomSubsectionsMutation } from "./viewingRooms/mutations/updateViewingRoomSubsections"
 
 const viewingRoomUnstitchedRootField = config.USE_UNSTITCHED_VIEWING_ROOM_SCHEMA
   ? {
@@ -272,6 +278,12 @@ const viewingRoomUnstitchedRootField = config.USE_UNSTITCHED_VIEWING_ROOM_SCHEMA
 const viewingRoomsMutations = config.USE_UNSTITCHED_VIEWING_ROOM_SCHEMA
   ? {
       createViewingRoom: createViewingRoomMutation,
+      deleteViewingRoom: deleteViewingRoomMutation,
+      publishViewingRoom: publishViewingRoomMutation,
+      unpublishViewingRoom: unpublishViewingRoomMutation,
+      updateViewingRoom: updateViewingRoomMutation,
+      updateViewingRoomArtworks: updateViewingRoomArtworksMutation,
+      updateViewingRoomSubsections: updateViewingRoomSubsectionsMutation,
     }
   : ({} as any)
 

src/schema/v2/viewingRoomSubsection.ts

@@ -28,6 +28,10 @@ export const ViewingRoomSubsectionType = new GraphQLObjectType<
       image: {
         type: GravityARImageType,
       },
+      imageURL: {
+        type: GraphQLString,
+        resolve: ({ image_url }) => image_url,
+      },
       title: {
         type: GraphQLString,
       },

src/schema/v2/viewingRooms/mutations/ARImageInput.ts

@@ -0,0 +1,10 @@
+import { GraphQLID, GraphQLInputObjectType, GraphQLNonNull } from "graphql"
+
+export const ARImageInputType = new GraphQLInputObjectType({
+  name: "ARImageInput",
+  fields: {
+    internalID: {
+      type: new GraphQLNonNull(GraphQLID),
+    },
+  },
+})

src/schema/v2/viewingRooms/mutations/__tests__/deleteViewingRoomMutation.test.ts

@@ -0,0 +1,47 @@
+import config from "config"
+import gql from "lib/gql"
+import { runAuthenticatedQuery } from "schema/v2/test/utils"
+
+describe("deleteViewingRoomMutation", () => {
+  const mockDeleteViewingRoomLoader = jest.fn()
+
+  const context = {
+    deleteViewingRoomLoader: mockDeleteViewingRoomLoader,
+  }
+
+  const viewingRoomData = {
+    id: "viewing-room-id",
+  }
+
+  beforeAll(() => {
+    config.USE_UNSTITCHED_VIEWING_ROOM_SCHEMA = true
+  })
+
+  afterAll(() => {
+    config.USE_UNSTITCHED_VIEWING_ROOM_SCHEMA = false
+  })
+
+  beforeEach(() => {
+    mockDeleteViewingRoomLoader.mockResolvedValue(
+      Promise.resolve(viewingRoomData)
+    )
+  })
+
+  afterEach(() => {
+    mockDeleteViewingRoomLoader.mockReset()
+  })
+
+  const mutation = gql`
+    mutation {
+      deleteViewingRoom(input: { viewingRoomID: "viewing-room-id" }) {
+        __typename
+      }
+    }
+  `
+
+  it("correctly calls the deleteViewingRoomLoader", async () => {
+    await runAuthenticatedQuery(mutation, context)
+
+    expect(mockDeleteViewingRoomLoader).toHaveBeenCalledWith("viewing-room-id")
+  })
+})

src/schema/v2/viewingRooms/mutations/__tests__/publishViewingRoomMutation.test.ts

@@ -0,0 +1,52 @@
+import config from "config"
+import gql from "lib/gql"
+import { runAuthenticatedQuery } from "schema/v2/test/utils"
+
+describe("publishViewingRoomMutation", () => {
+  const mockUpdateViewingRoomLoader = jest.fn()
+
+  const context = {
+    updateViewingRoomLoader: mockUpdateViewingRoomLoader,
+  }
+
+  const viewingRoomData = {
+    id: "viewing-room-id",
+  }
+
+  beforeAll(() => {
+    config.USE_UNSTITCHED_VIEWING_ROOM_SCHEMA = true
+  })
+
+  afterAll(() => {
+    config.USE_UNSTITCHED_VIEWING_ROOM_SCHEMA = false
+  })
+
+  beforeEach(() => {
+    mockUpdateViewingRoomLoader.mockResolvedValue(
+      Promise.resolve(viewingRoomData)
+    )
+  })
+
+  afterEach(() => {
+    mockUpdateViewingRoomLoader.mockReset()
+  })
+
+  const mutation = gql`
+    mutation {
+      publishViewingRoom(input: { viewingRoomID: "viewing-room-id" }) {
+        __typename
+      }
+    }
+  `
+
+  it("correctly calls the updateViewingRoomLoader", async () => {
+    await runAuthenticatedQuery(mutation, context)
+
+    expect(mockUpdateViewingRoomLoader).toHaveBeenCalledWith(
+      "viewing-room-id",
+      {
+        published: true,
+      }
+    )
+  })
+})