Merge pull request #3078 from OpenNeuroOrg/subscription-cleanup

chore(app): Cleanup unused GraphQL subscription hooks
OpenNeuroOrg · Jun 12, 2024 · 4e7fee9 · 4e7fee9
2 parents ebb4e21 + 024646e
commit 4e7fee9
Show file tree

Hide file tree

Showing 8 changed files with 0 additions and 302 deletions.
diff --git a/...rc/scripts/datalad/subscriptions/__tests__/__snapshots__/files-subscription.spec.jsx.snap b/...rc/scripts/datalad/subscriptions/__tests__/__snapshots__/files-subscription.spec.jsx.snap
diff --git a/...ges/openneuro-app/src/scripts/datalad/subscriptions/__tests__/files-subscription.spec.jsx b/...ges/openneuro-app/src/scripts/datalad/subscriptions/__tests__/files-subscription.spec.jsx
diff --git a/packages/openneuro-app/src/scripts/datalad/subscriptions/files-subscription.jsx b/packages/openneuro-app/src/scripts/datalad/subscriptions/files-subscription.jsx
diff --git a/packages/openneuro-app/src/scripts/datalad/subscriptions/useDatasetDeletedSubscription.jsx b/packages/openneuro-app/src/scripts/datalad/subscriptions/useDatasetDeletedSubscription.jsx
diff --git a/packages/openneuro-app/src/scripts/datalad/subscriptions/useDraftSubscription.js b/packages/openneuro-app/src/scripts/datalad/subscriptions/useDraftSubscription.js
diff --git a/packages/openneuro-app/src/scripts/datalad/subscriptions/usePermissionsSubscription.js b/packages/openneuro-app/src/scripts/datalad/subscriptions/usePermissionsSubscription.js
diff --git a/packages/openneuro-app/src/scripts/datalad/subscriptions/useSnapshotsUpdatedSubscriptions.js b/packages/openneuro-app/src/scripts/datalad/subscriptions/useSnapshotsUpdatedSubscriptions.js
diff --git a/packages/openneuro-app/src/scripts/dataset/dataset-query.jsx b/packages/openneuro-app/src/scripts/dataset/dataset-query.jsx
@@ -8,14 +8,6 @@ import { Loading } from "@openneuro/components/loading"
 import DatasetQueryContext from "../datalad/dataset/dataset-query-context.js"
 import DatasetContext from "../datalad/dataset/dataset-context.js"
 import DatasetRoutes from "./dataset-routes"
-import FilesSubscription from "../datalad/subscriptions/files-subscription.jsx"
-import usePermissionsSubscription from "../datalad/subscriptions/usePermissionsSubscription"
-import useSnapshotsUpdatedSubscriptions from "../datalad/subscriptions/useSnapshotsUpdatedSubscriptions"
-import useDatasetDeletedSubscription, {
-  datasetDeletedToast,
-} from "../datalad/subscriptions/useDatasetDeletedSubscription.jsx"
-import useDraftSubscription from "../datalad/subscriptions/useDraftSubscription.js"
-
 import ErrorBoundary, {
   ErrorBoundaryAssertionFailureException,
 } from "../errors/errorBoundary.jsx"
@@ -41,15 +33,6 @@ export const DatasetQueryHook = ({ datasetId, draft }) => {
       nextFetchPolicy: "cache-first",
     },
   )
-  usePermissionsSubscription([datasetId])
-  useSnapshotsUpdatedSubscriptions(datasetId)
-  useDatasetDeletedSubscription([datasetId], ({ data: subData }) => {
-    if (subData && subData.datasetDeleted === datasetId) {
-      navigate("/dashboard/datasets")
-      datasetDeletedToast(datasetId, data?.dataset?.draft?.description?.Name)
-    }
-  })
-  useDraftSubscription(datasetId)
 
   if (error) {
     if (error.message === "You do not have access to read this dataset.") {
@@ -94,7 +77,6 @@ export const DatasetQueryHook = ({ datasetId, draft }) => {
           }}
         >
           <DatasetRoutes dataset={data.dataset} />
-          <FilesSubscription datasetId={datasetId} />
         </DatasetQueryContext.Provider>
       </ErrorBoundary>
     </DatasetContext.Provider>