mirror of
https://github.com/supabase/supabase.git
synced 2026-09-06 09:59:03 +08:00
Follow-up to #48344: collapses the two resolution paths for the Admonition module into one. `src/admonition.tsx` was a back-compat shim re-exporting `src/Admonition/`. Two ways to resolve one module is exactly what produced the macOS self-import bug fixed in #48344, and the local typecheck errors that #48374 worked around. This removes the shim and standardizes on the PascalCase subpath, matching every other export in the package. **Changed:** - Codemodded all 246 `ui-patterns/admonition` imports to `ui-patterns/Admonition` (240 `.tsx`, 5 `.mdx`, 1 `.ts` across studio, docs, www, design-system, and lite-studio) - Pointed the 5 internal `'../admonition'` imports back at the `'../Admonition'` directory **Removed:** - `packages/ui-patterns/src/admonition.tsx`, and its `./admonition` entry in the exports map (regenerated with `pnpm gen:exports`) ## To test - `grep -r "ui-patterns/admonition" --include='*.ts*'` → no hits - `pnpm test:case-hazards` → passes - `pnpm typecheck` → all 15 tasks green - `pnpm --filter studio run lint:ratchet` → passes - `pnpm --filter ui-patterns vitest run src/Admonition` → 11 tests pass <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit * **Bug Fixes** * Standardized Admonition component imports across the application and documentation. * Improved compatibility with case-sensitive environments by using the canonical component path. * Removed the legacy Admonition import entry point. <!-- end of auto-generated comment: release notes by coderabbit.ai --> --------- Co-authored-by: Alaister Young <10985857+alaister@users.noreply.github.com>
149 lines
4.7 KiB
TypeScript
149 lines
4.7 KiB
TypeScript
import Link from 'next/link'
|
|
import { useMemo } from 'react'
|
|
import {
|
|
BreadcrumbItem,
|
|
BreadcrumbLink,
|
|
BreadcrumbList,
|
|
BreadcrumbPage,
|
|
BreadcrumbSeparator,
|
|
NavMenu,
|
|
NavMenuItem,
|
|
} from 'ui'
|
|
import { Admonition } from 'ui-patterns/Admonition'
|
|
import { PageContainer } from 'ui-patterns/PageContainer'
|
|
import {
|
|
PageHeader,
|
|
PageHeaderBreadcrumb,
|
|
PageHeaderDescription,
|
|
PageHeaderIcon,
|
|
PageHeaderMeta,
|
|
PageHeaderNavigationTabs,
|
|
PageHeaderSummary,
|
|
PageHeaderTitle,
|
|
} from 'ui-patterns/PageHeader'
|
|
import { PageSection, PageSectionContent } from 'ui-patterns/PageSection'
|
|
import { GenericSkeletonLoader, ShimmeringLoader } from 'ui-patterns/ShimmeringLoader'
|
|
|
|
import { IntegrationLogo } from '@/components/interfaces/Integrations/Integration/IntegrationLogo'
|
|
import { InstallOAuthIntegrationButton } from '@/components/interfaces/Integrations/Integration/IntegrationOverviewTabV2/InstallIntegrationSheet/InstallOAuthIntegrationButton'
|
|
import { useIntegrationDetail } from '@/components/interfaces/Integrations/Landing/useIntegrationDetail'
|
|
import { UnknownInterface } from '@/components/ui/UnknownInterface'
|
|
|
|
const LegacyIntegrationPage = () => {
|
|
const {
|
|
ref,
|
|
id,
|
|
isReady,
|
|
isWrapperBlocked,
|
|
pageTitle,
|
|
pageSubTitle,
|
|
integration,
|
|
isAvailableLoading,
|
|
isInstalledLoading,
|
|
isIntegrationStatusLoading,
|
|
oauthIntegrationData,
|
|
tabs,
|
|
Component,
|
|
} = useIntegrationDetail()
|
|
|
|
const content = useMemo(() => {
|
|
if (!isReady || isInstalledLoading || isAvailableLoading) {
|
|
return (
|
|
<PageContainer size="full">
|
|
<PageSection>
|
|
<PageSectionContent>
|
|
<GenericSkeletonLoader />
|
|
</PageSectionContent>
|
|
</PageSection>
|
|
</PageContainer>
|
|
)
|
|
} else if (!Component || !id || !integration) {
|
|
return (
|
|
<PageContainer size="full">
|
|
<PageSection>
|
|
<PageSectionContent>
|
|
<Admonition type="warning" title="This integration is not currently available">
|
|
Please try again later or contact support if the problem persists.
|
|
</Admonition>
|
|
</PageSectionContent>
|
|
</PageSection>
|
|
</PageContainer>
|
|
)
|
|
} else {
|
|
return <Component />
|
|
}
|
|
}, [isReady, isInstalledLoading, isAvailableLoading, id, integration, Component])
|
|
|
|
if (!isReady) return null
|
|
if (isWrapperBlocked) return <UnknownInterface urlBack={`/project/${ref}/integrations`} />
|
|
|
|
return (
|
|
<>
|
|
<PageHeader size="full">
|
|
<PageHeaderBreadcrumb className="mx-auto w-full">
|
|
<BreadcrumbList>
|
|
<BreadcrumbItem>
|
|
<BreadcrumbLink asChild>
|
|
<Link href={`/project/${ref}/integrations`}>Integrations</Link>
|
|
</BreadcrumbLink>
|
|
</BreadcrumbItem>
|
|
<BreadcrumbSeparator />
|
|
<BreadcrumbItem>
|
|
<BreadcrumbPage>{integration?.name || 'Integration not found'}</BreadcrumbPage>
|
|
</BreadcrumbItem>
|
|
</BreadcrumbList>
|
|
</PageHeaderBreadcrumb>
|
|
|
|
{isAvailableLoading ? (
|
|
<PageHeaderMeta className="mx-auto w-full">
|
|
<PageHeaderSummary>
|
|
<PageHeaderTitle>
|
|
<ShimmeringLoader className="w-64 py-4" />
|
|
</PageHeaderTitle>
|
|
<PageHeaderDescription>
|
|
<ShimmeringLoader />
|
|
</PageHeaderDescription>
|
|
</PageHeaderSummary>
|
|
</PageHeaderMeta>
|
|
) : (
|
|
<PageHeaderMeta className="mx-auto w-full">
|
|
{integration && (
|
|
<PageHeaderIcon>
|
|
<IntegrationLogo integration={integration} size="w-14 h-14" />
|
|
</PageHeaderIcon>
|
|
)}
|
|
<PageHeaderSummary className="gap-y-0.5">
|
|
<PageHeaderTitle>{pageTitle}</PageHeaderTitle>
|
|
<PageHeaderDescription>{pageSubTitle}</PageHeaderDescription>
|
|
</PageHeaderSummary>
|
|
|
|
{integration?.type === 'oauth' && (
|
|
<InstallOAuthIntegrationButton
|
|
integration={integration}
|
|
data={oauthIntegrationData}
|
|
isLoading={isIntegrationStatusLoading}
|
|
/>
|
|
)}
|
|
</PageHeaderMeta>
|
|
)}
|
|
|
|
{tabs.length > 0 && (
|
|
<PageHeaderNavigationTabs className="mx-auto w-full">
|
|
<NavMenu>
|
|
{tabs.map((tab) => (
|
|
<NavMenuItem key={tab.href} active={tab.active}>
|
|
<Link href={tab.href}>{tab.label}</Link>
|
|
</NavMenuItem>
|
|
))}
|
|
</NavMenu>
|
|
</PageHeaderNavigationTabs>
|
|
)}
|
|
</PageHeader>
|
|
|
|
<div className="flex-1 min-h-0 mx-auto w-full">{content}</div>
|
|
</>
|
|
)
|
|
}
|
|
|
|
export default LegacyIntegrationPage
|