fix(hub-common): make getUser optional (#1722)

Esri · Nov 8, 2024 · b3531bd · b3531bd
1 parent af7a95f
commit b3531bd
Show file tree

Hide file tree

Showing 3 changed files with 106 additions and 74 deletions.
diff --git a/package-lock.json b/package-lock.json
diff --git a/packages/common/src/search/_internal/hubEventsHelpers/eventToSearchResult.ts b/packages/common/src/search/_internal/hubEventsHelpers/eventToSearchResult.ts
@@ -1,4 +1,4 @@
-import { getUser } from "@esri/arcgis-rest-portal";
+import { getUser, IUser } from "@esri/arcgis-rest-portal";
 import { IHubSearchOptions } from "../../types/IHubSearchOptions";
 import { IHubSearchResult } from "../../types/IHubSearchResult";
 import { IEvent } from "../../../events/api/orval/api/orval-events";
@@ -17,10 +17,13 @@ export async function eventToSearchResult(
   event: IEvent,
   options: IHubSearchOptions
 ): Promise<IHubSearchResult> {
-  const ownerUser = await getUser({
-    username: event.creator.username,
-    ...options.requestOptions,
-  });
+  let ownerUser: IUser;
+  if (options.include?.includes("ownerUser")) {
+    ownerUser = await getUser({
+      username: event.creator.username,
+      ...options.requestOptions,
+    });
+  }
   const result = {
     access: event.access.toLowerCase() as AccessLevel,
     id: event.id,