fix(react-components): don't rely on revisionId in CogniteCADRevision

It's not always there for the moment, apparently
cognitedata · Sep 6, 2024 · 60ba284 · 60ba284
1 parent a18a7bd
commit 60ba284
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 12 deletions.
diff --git a/react-components/src/data-providers/core-dm-provider/getDMSRevision.ts b/react-components/src/data-providers/core-dm-provider/getDMSRevision.ts
@@ -12,9 +12,10 @@ export async function getDMSRevision(
   revisionId: number,
   fdmSdk: FdmSDK
 ): Promise<NodeItem<CogniteCADRevisionProperties>> {
+  const revisionExternalId = `cog_3d_revision_${revisionId}`;
   const query = {
     ...revisionQuery,
-    parameters: { modelReference: restrictToDmsId(model), revisionId }
+    parameters: { modelReference: restrictToDmsId(model), revisionExternalId }
   } as const satisfies QueryRequest;
 
   const result = await fdmSdk.queryNodesAndEdges<

diff --git a/react-components/src/data-providers/core-dm-provider/revisionQuery.ts b/react-components/src/data-providers/core-dm-provider/revisionQuery.ts
@@ -2,11 +2,7 @@
  * Copyright 2024 Cognite AS
  */
 import { type QueryRequest } from '@cognite/sdk/dist/src';
-import {
-  COGNITE_3D_REVISION_SOURCE,
-  COGNITE_CAD_REVISION_SOURCE,
-  CORE_DM_3D_CONTAINER_SPACE
-} from './dataModels';
+import { COGNITE_3D_REVISION_SOURCE, CORE_DM_3D_CONTAINER_SPACE } from './dataModels';
 import { cogniteCadRevisionSourceWithProperties } from './cogniteCadRevisionSourceWithProperties';
 
 export const revisionQuery = {
@@ -27,12 +23,8 @@ export const revisionQuery = {
             },
             {
               equals: {
-                property: [
-                  CORE_DM_3D_CONTAINER_SPACE,
-                  COGNITE_CAD_REVISION_SOURCE.externalId,
-                  'revisionId'
-                ],
-                value: { parameter: 'revisionId' }
+                property: ['node', 'externalId'],
+                value: { parameter: 'revisionExternalId' }
               }
             }
           ]