fix(react): update app redirect url to new dashboard (#377)

Author: binoy14

packages/react/src/components/SanityApp.test.tsx

@@ -1,4 +1,4 @@
-import {AuthStateType} from '@sanity/sdk'
+import {AuthStateType, type SanityConfig} from '@sanity/sdk'
 import {render, screen} from '@testing-library/react'
 import {describe, expect, it, vi} from 'vitest'
 
@@ -142,4 +142,121 @@ describe('SanityApp', () => {
     // Config should be passed to SDKProvider in the same order
     expect(config).toEqual(legacyConfigs)
   })
+
+  it('handles iframe environment correctly', async () => {
+    // Mock window.self and window.top to simulate iframe environment
+    const originalTop = window.top
+    const originalSelf = window.self
+
+    const mockSanityConfig: SanityConfig = {
+      projectId: 'test-project',
+      dataset: 'test-dataset',
+    }
+
+    const mockTop = {}
+    Object.defineProperty(window, 'top', {
+      value: mockTop,
+      writable: true,
+    })
+    Object.defineProperty(window, 'self', {
+      value: window,
+      writable: true,
+    })
+
+    render(
+      <SanityApp config={[mockSanityConfig]} fallback={<div>Fallback</div>}>
+        <div>Test Child</div>
+      </SanityApp>,
+    )
+
+    // Wait for 1 second
+    await new Promise((resolve) => setTimeout(resolve, 1010))
+
+    // Add assertions based on your iframe-specific behavior
+    expect(window.location.href).toBe('http://localhost:3000/')
+
+    // Clean up the mock
+    Object.defineProperty(window, 'top', {
+      value: originalTop,
+      writable: true,
+    })
+    Object.defineProperty(window, 'self', {
+      value: originalSelf,
+      writable: true,
+    })
+  })
+
+  it('redirects to core if not inside iframe and not local url', async () => {
+    const originalLocation = window.location
+
+    const mockLocation = {
+      replace: vi.fn(),
+      href: 'http://sanity-test.app',
+    }
+
+    const mockSanityConfig: SanityConfig = {
+      projectId: 'test-project',
+      dataset: 'test-dataset',
+    }
+
+    Object.defineProperty(window, 'location', {
+      value: mockLocation,
+      writable: true,
+    })
+
+    render(
+      <SanityApp config={[mockSanityConfig]} fallback={<div>Fallback</div>}>
+        <div>Test Child</div>
+      </SanityApp>,
+    )
+
+    // Wait for 1 second
+    await new Promise((resolve) => setTimeout(resolve, 1010))
+
+    // Add assertions based on your iframe-specific behavior
+    expect(mockLocation.replace).toHaveBeenCalledWith('https://sanity.io/welcome')
+
+    // Clean up the mock
+    Object.defineProperty(window, 'location', {
+      value: originalLocation,
+      writable: true,
+    })
+  })
+
+  it('does not redirect to core if not inside iframe and local url', async () => {
+    const originalLocation = window.location
+
+    const mockSanityConfig: SanityConfig = {
+      projectId: 'test-project',
+      dataset: 'test-dataset',
+    }
+
+    const mockLocation = {
+      replace: vi.fn(),
+      href: 'http://localhost:3000',
+    }
+
+    Object.defineProperty(window, 'location', {
+      value: mockLocation,
+      writable: true,
+    })
+
+    render(
+      <SanityApp config={[mockSanityConfig]} fallback={<div>Fallback</div>}>
+        <div>Test Child</div>
+      </SanityApp>,
+    )
+
+    // Wait for 1 second
+    await new Promise((resolve) => setTimeout(resolve, 1010))
+
+    // Add assertions based on your iframe-specific behavior
+    expect(mockLocation.replace).not.toHaveBeenCalled()
+
+    // Clean up the mock
+    Object.defineProperty(window, 'location', {
+      value: originalLocation,
+      writable: true,
+    })
+  })
 })

packages/react/src/components/SanityApp.tsx

@@ -15,7 +15,7 @@ export interface SanityAppProps {
   fallback: React.ReactNode
 }
 
-const CORE_URL = 'https://core.sanity.io'
+const REDIRECT_URL = 'https://sanity.io/welcome'
 
 /**
  * @public
@@ -86,8 +86,8 @@ export function SanityApp({
       // If the app is not running in an iframe and is not a local url, redirect to core.
       timeout = setTimeout(() => {
         // eslint-disable-next-line no-console
-        console.warn('Redirecting to core', CORE_URL)
-        window.location.replace(CORE_URL)
+        console.warn('Redirecting to core', REDIRECT_URL)
+        window.location.replace(REDIRECT_URL)
       }, 1000)
     }
     return () => clearTimeout(timeout)