fix(structure): add support to force a version for the pane provider (#…

…8176)
sanity-io · Jan 3, 2025 · e9a5002 · e9a5002
1 parent b512378
commit e9a5002
Show file tree

Hide file tree

Showing 3 changed files with 32 additions and 23 deletions.
diff --git a/packages/sanity/src/structure/panes/document/DocumentEventsPane.tsx b/packages/sanity/src/structure/panes/document/DocumentEventsPane.tsx
@@ -1,3 +1,4 @@
+import {type ReleaseId} from '@sanity/client'
 import {useMemo} from 'react'
 import {
   EventsProvider,
@@ -24,7 +25,8 @@ export const DocumentEventsPane = (props: DocumentPaneProviderProps) => {
 
   const {selectedPerspectiveName} = usePerspective()
   const {archivedReleases} = useReleases()
-  const {rev, since, historyVersion} = params
+  const {rev, since} = params
+  const historyVersion = params.historyVersion as ReleaseId | undefined
 
   const documentId = useMemo(() => {
     if (
@@ -69,9 +71,24 @@ export const DocumentEventsPane = (props: DocumentPaneProviderProps) => {
 
   const value = useMemo(() => eventsStore, [eventsStore])
 
+  const forcedProviderVersion = useMemo(() => {
+    if (historyVersion) {
+      return {
+        selectedPerspectiveName: historyVersion,
+        selectedReleaseId: historyVersion,
+        isReleaseLocked: true,
+      }
+    }
+    return undefined
+  }, [historyVersion])
+
   return (
     <EventsProvider value={value}>
-      <DocumentPaneProvider {...props} historyStore={historyStoreProps} />
+      <DocumentPaneProvider
+        {...props}
+        historyStore={historyStoreProps}
+        forcedVersion={forcedProviderVersion}
+      />
     </EventsProvider>
   )
 }
diff --git a/packages/sanity/src/structure/panes/document/DocumentPaneProvider.tsx b/packages/sanity/src/structure/panes/document/DocumentPaneProvider.tsx
@@ -1,5 +1,4 @@
 /* eslint-disable camelcase */
-import {type ReleaseId} from '@sanity/client'
 import {isActionEnabled} from '@sanity/schema/_internal'
 import {useTelemetry} from '@sanity/telemetry/react'
 import {
@@ -81,7 +80,7 @@ interface DocumentPaneProviderProps extends DocumentPaneProviderWrapperProps {
  */
 // eslint-disable-next-line complexity, max-statements
 export const DocumentPaneProvider = memo((props: DocumentPaneProviderProps) => {
-  const {children, index, pane, paneKey, onFocusPath, matchGlobalVersion = true} = props
+  const {children, index, pane, paneKey, onFocusPath, forcedVersion} = props
   const schema = useSchema()
   const templates = useTemplates()
   const {setDocumentMeta} = useCopyPaste()
@@ -116,20 +115,8 @@ export const DocumentPaneProvider = memo((props: DocumentPaneProviderProps) => {
   const perspective = usePerspective()
 
   const {isReleaseLocked, selectedReleaseId, selectedPerspectiveName} = useMemo(() => {
-    if (!matchGlobalVersion) {
-      return {
-        selectedPerspectiveName: undefined,
-        isReleaseLocked: false,
-        selectedReleaseId: undefined,
-      }
-    }
-    const historyVersion = params.historyVersion as ReleaseId | undefined
-    if (historyVersion) {
-      return {
-        selectedPerspectiveName: historyVersion,
-        selectedReleaseId: historyVersion,
-        isReleaseLocked: true,
-      }
+    if (forcedVersion) {
+      return forcedVersion
     }
     return {
       selectedPerspectiveName: perspective.selectedPerspectiveName,
@@ -139,8 +126,7 @@ export const DocumentPaneProvider = memo((props: DocumentPaneProviderProps) => {
         : false,
     }
   }, [
-    matchGlobalVersion,
-    params.historyVersion,
+    forcedVersion,
     perspective.selectedPerspectiveName,
     perspective.selectedReleaseId,
     perspective.selectedPerspective,

diff --git a/packages/sanity/src/structure/panes/types.ts b/packages/sanity/src/structure/panes/types.ts
@@ -1,3 +1,5 @@
+import {type ReleaseId} from '@sanity/client'
+
 import {type PaneNode} from '../types'
 
 export interface BaseStructureToolPaneProps<T extends PaneNode['type']> {
@@ -9,8 +11,12 @@ export interface BaseStructureToolPaneProps<T extends PaneNode['type']> {
   isActive?: boolean
   pane: Extract<PaneNode, {type: T}>
   /**
-   * Decides if the pane should check for the global version when determining the document id.
-   * default: true
+   * Allows to override the global version with a specific version or release.
+   * @beta
    */
-  matchGlobalVersion?: boolean
+  forcedVersion?: {
+    selectedPerspectiveName: ReleaseId | 'published' | undefined
+    isReleaseLocked: boolean
+    selectedReleaseId: ReleaseId | undefined
+  }
 }