mirror of
https://github.com/supabase/supabase.git
synced 2026-10-07 02:15:05 +03: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>
101 lines
3.9 KiB
TypeScript
101 lines
3.9 KiB
TypeScript
import { useParams } from 'common'
|
|
import { Lock } from 'lucide-react'
|
|
import { toast } from 'sonner'
|
|
import {
|
|
AlertDialog,
|
|
AlertDialogAction,
|
|
AlertDialogCancel,
|
|
AlertDialogContent,
|
|
AlertDialogDescription,
|
|
AlertDialogFooter,
|
|
AlertDialogHeader,
|
|
AlertDialogTitle,
|
|
} from 'ui'
|
|
import { Admonition } from 'ui-patterns/Admonition'
|
|
|
|
import { InlineLink } from '@/components/ui/InlineLink'
|
|
import { useAuthorizedAppRevokeMutation } from '@/data/oauth/authorized-app-revoke-mutation'
|
|
import type { AuthorizedApp } from '@/data/oauth/authorized-apps-query'
|
|
|
|
export interface RevokeAppModalProps {
|
|
selectedApp?: AuthorizedApp
|
|
/** Optional Organization slug override for routes without a `slug` param (e.g. project integrations). */
|
|
orgSlug?: string
|
|
onClose: () => void
|
|
}
|
|
|
|
export const RevokeAppModal = ({
|
|
selectedApp,
|
|
orgSlug: slugOverride,
|
|
onClose,
|
|
}: RevokeAppModalProps) => {
|
|
const { slug: slugParam } = useParams()
|
|
const orgSlug = slugOverride ?? slugParam
|
|
const { mutateAsync: revokeAuthorizedApp } = useAuthorizedAppRevokeMutation({
|
|
onSuccess: () => {
|
|
toast.success(`Successfully revoked the app "${selectedApp?.name}"`)
|
|
onClose()
|
|
},
|
|
})
|
|
|
|
const onConfirmDelete = async () => {
|
|
if (!orgSlug) return console.error('Organization slug is required')
|
|
if (!selectedApp?.id) return console.error('App ID is required')
|
|
await revokeAuthorizedApp({ orgSlug, id: selectedApp?.id })
|
|
}
|
|
|
|
return (
|
|
<AlertDialog open={selectedApp !== undefined} onOpenChange={onClose}>
|
|
<AlertDialogContent size="medium">
|
|
<AlertDialogHeader>
|
|
<AlertDialogTitle>{`Revoke access for ${selectedApp?.name}?`}</AlertDialogTitle>
|
|
<AlertDialogDescription>
|
|
<div className="flex flex-col space-y-4">
|
|
<Admonition
|
|
type="warning"
|
|
title="This action cannot be undone"
|
|
description={`${selectedApp?.name} will no longer have access to your organization's settings
|
|
and projects.`}
|
|
/>
|
|
<div className="space-y-5">
|
|
<div className="flex gap-2 text-sm">
|
|
<Lock size={14} className="shrink-0" />
|
|
<div>
|
|
<strong>Before you remove this app, consider:</strong>
|
|
<ul className="space-y-2 mt-2">
|
|
<li className="list-disc ml-4">
|
|
The application will no longer have access to your organization after being
|
|
revoked.
|
|
</li>
|
|
<li className="list-disc ml-4">
|
|
This will remove the application for all members in your organization.
|
|
</li>
|
|
<li className="list-disc ml-4">
|
|
Restoring access will require an organization administrator to re-authorize
|
|
the application.
|
|
</li>
|
|
<li className="list-disc ml-4">
|
|
The application may also have a <strong>Secret API key</strong> with access.
|
|
Navigate to{' '}
|
|
<InlineLink href={`/dashboard/project/_/integrations`}>
|
|
Integrations
|
|
</InlineLink>{' '}
|
|
on the project this app was installed, and remove any listed Secret API key
|
|
in the "Settings" tab of the integration to fully revoke its access.
|
|
</li>
|
|
</ul>
|
|
</div>
|
|
</div>
|
|
</div>
|
|
</div>
|
|
</AlertDialogDescription>
|
|
</AlertDialogHeader>
|
|
<AlertDialogFooter>
|
|
<AlertDialogCancel>Cancel</AlertDialogCancel>
|
|
<AlertDialogAction onClick={onConfirmDelete}>Confirm</AlertDialogAction>
|
|
</AlertDialogFooter>
|
|
</AlertDialogContent>
|
|
</AlertDialog>
|
|
)
|
|
}
|