mirror of
https://github.com/supabase/supabase.git
synced 2026-06-17 13:14:06 +08:00
## Problem Organization settings still uses the deprecated `Modal` for: - downgrading subscription - requesting feedback after downgrading - showing an alert about members limit - requesting feedback after upgrading - deleting a published OAuth app - showing preview of a new OAuth app - Revoking an OAuth app ## Solution - use `Dialog` instead <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit * **Refactor** * Replaced legacy modal UI with the app's modern dialog/alert-dialog components across billing and OAuth settings (upgrade/downgrade, exit survey, members-limit, delete/revoke, preview), keeping existing content and flows. * Confirm/cancel flows updated for more reliable async handling and clearer loading/disabled states during actions. <!-- review_stack_entry_start --> [](https://app.coderabbit.ai/change-stack/supabase/supabase/pull/46332?utm_source=github_walkthrough&utm_medium=github&utm_campaign=change_stack) <!-- review_stack_entry_end --> <!-- end of auto-generated comment: release notes by coderabbit.ai -->
76 lines
2.5 KiB
TypeScript
76 lines
2.5 KiB
TypeScript
import { useParams } from 'common'
|
|
import { Lock } from 'lucide-react'
|
|
import { toast } from 'sonner'
|
|
import {
|
|
AlertDialog,
|
|
AlertDialogAction,
|
|
AlertDialogCancel,
|
|
AlertDialogContent,
|
|
AlertDialogDescription,
|
|
AlertDialogFooter,
|
|
AlertDialogTitle,
|
|
} from 'ui'
|
|
import { Admonition } from 'ui-patterns'
|
|
|
|
import { useOAuthAppDeleteMutation } from '@/data/oauth/oauth-app-delete-mutation'
|
|
import type { OAuthApp } from '@/data/oauth/oauth-apps-query'
|
|
|
|
export interface DeleteAppModalProps {
|
|
selectedApp?: OAuthApp
|
|
onClose: () => void
|
|
}
|
|
|
|
export const DeleteAppModal = ({ selectedApp, onClose }: DeleteAppModalProps) => {
|
|
const { slug } = useParams()
|
|
const { mutateAsync: deleteOAuthApp } = useOAuthAppDeleteMutation({
|
|
onSuccess: () => {
|
|
toast.success(`Successfully deleted the app "${selectedApp?.name}"`)
|
|
onClose()
|
|
},
|
|
})
|
|
|
|
const onConfirmDelete = async () => {
|
|
if (!slug) return console.error('Slug is required')
|
|
if (!selectedApp?.id) return console.error('App ID is required')
|
|
await deleteOAuthApp({ slug, id: selectedApp?.id })
|
|
}
|
|
|
|
return (
|
|
<AlertDialog open={selectedApp !== undefined} onOpenChange={onClose}>
|
|
<AlertDialogContent size="medium">
|
|
<AlertDialogTitle>{`Confirm to delete ${selectedApp?.name}`}</AlertDialogTitle>
|
|
<AlertDialogDescription>
|
|
<div className="flex flex-col space-y-2">
|
|
<Admonition
|
|
type="warning"
|
|
title="This action cannot be undone"
|
|
description={`Deleting ${selectedApp?.name} will invalidate any access tokens from this application that
|
|
were authorized by users.`}
|
|
/>
|
|
<ul className="space-y-5">
|
|
<li className="flex gap-3 text-sm">
|
|
<Lock size={14} className="shrink-0" />
|
|
<div>
|
|
<strong>Before you remove this application, consider:</strong>
|
|
<ul className="space-y-2 mt-2">
|
|
<li className="list-disc ml-4">
|
|
No users are currently using this application. It will no longer be available
|
|
for use after deletion.
|
|
</li>
|
|
</ul>
|
|
</div>
|
|
</li>
|
|
</ul>
|
|
</div>
|
|
</AlertDialogDescription>
|
|
<AlertDialogFooter>
|
|
<AlertDialogCancel>Cancel</AlertDialogCancel>
|
|
<AlertDialogAction onClick={onConfirmDelete} variant="danger">
|
|
Confirm
|
|
</AlertDialogAction>
|
|
</AlertDialogFooter>
|
|
</AlertDialogContent>
|
|
</AlertDialog>
|
|
)
|
|
}
|