diff --git a/apps/docs/content/guides/database/functions.mdx b/apps/docs/content/guides/database/functions.mdx index 8fb0158569b..8186c100eb4 100644 --- a/apps/docs/content/guides/database/functions.mdx +++ b/apps/docs/content/guides/database/functions.mdx @@ -62,7 +62,7 @@ At it's most basic a function has the following parts: -When naming your functions, please make the name of the function unique as overloaded functions are not supported. +When naming your functions, make the name of the function unique as overloaded functions are not supported. diff --git a/apps/docs/content/guides/database/postgres/setup-replication-external.mdx b/apps/docs/content/guides/database/postgres/setup-replication-external.mdx index 0e74c9df844..a29c9156df7 100644 --- a/apps/docs/content/guides/database/postgres/setup-replication-external.mdx +++ b/apps/docs/content/guides/database/postgres/setup-replication-external.mdx @@ -49,7 +49,7 @@ To copy data from before the slot was created, set `copy_data` to `true`. -4. Add all the tables that you want replicated to the publication. +4. Now we'll go back to the Supabase DB and add all the tables that you want replicated to the publication. ```sql ALTER PUBLICATION example_pub ADD TABLE example_table; diff --git a/apps/docs/content/guides/functions/auth.mdx b/apps/docs/content/guides/functions/auth.mdx index 34ec98d4e36..0ade89bb3ac 100644 --- a/apps/docs/content/guides/functions/auth.mdx +++ b/apps/docs/content/guides/functions/auth.mdx @@ -19,9 +19,11 @@ Deno.serve(async (req: Request) => { const supabaseClient = createClient( Deno.env.get('SUPABASE_URL') ?? '', Deno.env.get('SUPABASE_ANON_KEY') ?? '', + // Create client with Auth context of the user that called the function. + // This way your row-level-security (RLS) policies are applied. { global: { - headers: { Authorization: req.headers.get('Authorization') }, + headers: { Authorization: req.headers.get('Authorization')! }, }, } ); @@ -38,7 +40,7 @@ Importantly, this is done _inside_ the `Deno.serve()` callback argument, so that ## Fetching the user -After initializing a Supabase client with the Auth context, you can use `getUser()` to fetch the user object, and run queries in the context of the user with [Row Level Security (RLS)](/docs/guides/database/postgres/row-level-security) policies enforced. +By getting the JWT from the `Authorization` header, you can provide the token to `getUser()` to fetch the user object to obtain metadata for the logged in user. ```js import { createClient } from 'npm:@supabase/supabase-js@2' @@ -81,9 +83,11 @@ Deno.serve(async (req: Request) => { const supabaseClient = createClient( Deno.env.get('SUPABASE_URL') ?? '', Deno.env.get('SUPABASE_ANON_KEY') ?? '', + // Create client with Auth context of the user that called the function. + // This way your row-level-security (RLS) policies are applied. { global: { - headers: { Authorization: req.headers.get('Authorization') }, + headers: { Authorization: req.headers.get('Authorization')! }, }, } ); diff --git a/apps/docs/content/guides/getting-started/mcp.mdx b/apps/docs/content/guides/getting-started/mcp.mdx index c9f4e45f947..fa878830241 100644 --- a/apps/docs/content/guides/getting-started/mcp.mdx +++ b/apps/docs/content/guides/getting-started/mcp.mdx @@ -95,6 +95,10 @@ For more info on using MCP in VS Code, see the [Copilot documentation](https://c ### Claude code +You can add the Supabase MCP server to Claude Code in two ways: + +#### Option 1: Project-scoped server (via .mcp.json file) + 1. Create a `.mcp.json` file in your project root if it doesn't exist. 1. Add the following configuration: @@ -104,6 +108,18 @@ For more info on using MCP in VS Code, see the [Copilot documentation](https://c 1. Restart [Claude code](https://claude.ai/code) to apply the new configuration. +#### Option 2: Locally-scoped server (via CLI command) + +You can also add the Supabase MCP server as a locally-scoped server, which will only be available to you in the current project: + +1. Run the following command in your terminal: + + ```bash + claude mcp add supabase -s local -e SUPABASE_ACCESS_TOKEN=your_token_here npx -y @supabase/mcp-server-supabase@latest + ``` + +Locally-scoped servers take precedence over project-scoped servers with the same name and are stored in your project-specific user settings. + ### Next steps Your AI tool is now connected to Supabase using MCP. Try asking your AI assistant to create a new project, create a table, or fetch project config. diff --git a/apps/docs/content/guides/platform/manage-your-usage/egress.mdx b/apps/docs/content/guides/platform/manage-your-usage/egress.mdx index 9f979e821ee..71a893036af 100644 --- a/apps/docs/content/guides/platform/manage-your-usage/egress.mdx +++ b/apps/docs/content/guides/platform/manage-your-usage/egress.mdx @@ -45,7 +45,7 @@ Data pushed to clients via Supabase Realtime for subscribed events. ### Shared pooler Egress -Data sent to the client when using the shared connection pooler (Supavisor) to access your database. WHen using the shared connection pooler, we do not count database egress, as this would otherwise count double (Database -> Shared Pooler + Shared Pooler -> Client). +Data sent to the client when using the shared connection pooler (Supavisor) to access your database. When using the shared connection pooler, we do not count database egress, as this would otherwise count double (Database -> Shared Pooler + Shared Pooler -> Client). **Example:** You are using our [shared connection pooler](https://supabase.com/docs/guides/database/connecting-to-postgres#shared-pooler) and you query a list of invoices in your backend. The data returned from that query is contributing to Shared Pooler Egress. diff --git a/apps/docs/features/ui/CodeBlock/CodeBlock.client.tsx b/apps/docs/features/ui/CodeBlock/CodeBlock.client.tsx index da8a74b65f2..9b3ca95161f 100644 --- a/apps/docs/features/ui/CodeBlock/CodeBlock.client.tsx +++ b/apps/docs/features/ui/CodeBlock/CodeBlock.client.tsx @@ -4,7 +4,7 @@ import { Check, Copy } from 'lucide-react' import { type MouseEvent, useCallback, useEffect, useState } from 'react' import { type ThemedToken } from 'shiki' import { type NodeHover } from 'twoslash' -import { cn, Tooltip, TooltipContent, TooltipTrigger } from 'ui' +import { cn, copyToClipboard, Tooltip, TooltipContent, TooltipTrigger } from 'ui' export function AnnotatedSpan({ token, @@ -89,13 +89,10 @@ export function CodeCopyButton({ className, content }: { className?: string; con const [copied, setCopied] = useState(false) const handleCopy = async () => { - try { - await navigator.clipboard.writeText(content) + copyToClipboard(content, () => { setCopied(true) setTimeout(() => setCopied(false), 1000) - } catch (error) { - console.error('Failed to copy text: ', error) - } + }) } return ( diff --git a/apps/docs/public/humans.txt b/apps/docs/public/humans.txt index 18b5d9a3207..1dba617c869 100644 --- a/apps/docs/public/humans.txt +++ b/apps/docs/public/humans.txt @@ -45,6 +45,7 @@ Greg Richardson Guilherme Souza Haydn Maley Hieu Pham +Ignacio Dobronich Illia Basalaiev Inian P Ivan Vasilov diff --git a/apps/docs/scripts/helpers.mdx.ts b/apps/docs/scripts/helpers.mdx.ts index cbcbfb8f709..c6d9ce672ef 100644 --- a/apps/docs/scripts/helpers.mdx.ts +++ b/apps/docs/scripts/helpers.mdx.ts @@ -1,4 +1,4 @@ -import { createHash } from 'crypto' +import { createHash } from 'node:crypto' import { ObjectExpression } from 'estree' import GithubSlugger from 'github-slugger' import matter from 'gray-matter' diff --git a/apps/docs/scripts/search/sources/index.ts b/apps/docs/scripts/search/sources/index.ts index 5f712ded87f..c46c46a7986 100644 --- a/apps/docs/scripts/search/sources/index.ts +++ b/apps/docs/scripts/search/sources/index.ts @@ -5,6 +5,7 @@ import { type GitHubDiscussionSource, fetchDiscussions, } from './github-discussion.js' +import { LintWarningsGuideLoader, type LintWarningsGuideSource } from './lint-warnings-guide.js' import { MarkdownLoader, type MarkdownSource } from './markdown.js' import { IntegrationLoader, type IntegrationSource, fetchPartners } from './partner-integrations.js' import { @@ -26,6 +27,7 @@ export type SearchSource = | CliReferenceSource | GitHubDiscussionSource | IntegrationSource + | LintWarningsGuideSource export async function fetchGuideSources() { return ( @@ -122,12 +124,23 @@ export async function fetchCliLibReferenceSource() { ).load() } +export async function fetchLintWarningsGuideSources() { + return new LintWarningsGuideLoader( + 'guide', + '/guides/database/database-advisors', + 'supabase', + 'splinter', + 'main', + 'docs' + ).load() +} + /** * Fetches all the sources we want to index for search */ export async function fetchAllSources() { const guideSources = fetchGuideSources() - + const lintWarningsGuideSources = fetchLintWarningsGuideSources() const openApiReferenceSource = fetchOpenApiReferenceSource() const jsLibReferenceSource = fetchJsLibReferenceSource() const dartLibReferenceSource = fetchDartLibReferenceSource() @@ -164,6 +177,7 @@ export async function fetchAllSources() { const sources: SearchSource[] = ( await Promise.all([ guideSources, + lintWarningsGuideSources, openApiReferenceSource, jsLibReferenceSource, dartLibReferenceSource, diff --git a/apps/docs/scripts/search/sources/lint-warnings-guide.ts b/apps/docs/scripts/search/sources/lint-warnings-guide.ts new file mode 100644 index 00000000000..61d4f5e4e34 --- /dev/null +++ b/apps/docs/scripts/search/sources/lint-warnings-guide.ts @@ -0,0 +1,116 @@ +import { Octokit } from '@octokit/core' +import { BaseLoader, BaseSource } from './base.js' +import { createHash } from 'node:crypto' + +const getBasename = (path: string) => path.split('/').at(-1)!.replace(/\.md$/, '') + +export class LintWarningsGuideLoader extends BaseLoader { + type = 'markdown' as const + + constructor( + source: string, + path: string, + public org: string, + public repo: string, + public branch: string, + public docsDir: string + ) { + super(source, path) + } + + async load() { + const octokit = new Octokit() + + const response = await octokit.request('GET /repos/{owner}/{repo}/contents/{path}', { + owner: this.org, + repo: this.repo, + path: this.docsDir, + ref: this.branch, + headers: { + 'X-GitHub-Api-Version': '2022-11-28', + }, + }) + + if (response.status >= 400) { + throw Error(`Could not get contents of repo ${this.org}/${this.repo}`) + } + + if (!Array.isArray(response.data)) { + throw Error( + 'Reading a directory, not a file. Should not reach this, solely to appease Typescript.' + ) + } + + const lintsList = response.data.filter(({ path }) => /docs\/\d+.+\.md$/.test(path)) + + // Fetch all lint files and combine them into a single guide + const lints = await Promise.all( + lintsList.map(async ({ path }) => { + const fileResponse = await fetch( + `https://raw.githubusercontent.com/${this.org}/${this.repo}/${this.branch}/${path}` + ) + + if (fileResponse.status >= 400) { + throw Error(`Could not get contents of file ${this.org}/${this.repo}/${path}`) + } + + const content = await fileResponse.text() + const basename = getBasename(path) + + return { + path: basename, + content, + originalPath: path, + } + }) + ) + + // Create a separate source for each lint file + return lints.map( + (lint) => + new LintWarningsGuideSource( + this.source, + `${this.path}?queryGroups=lint&lint=${lint.path}`, + lint + ) + ) + } +} + +export class LintWarningsGuideSource extends BaseSource { + type = 'markdown' as const + + constructor( + source: string, + path: string, + public lint: { + path: string + content: string + originalPath: string + } + ) { + super(source, path) + } + + process() { + this.checksum = createHash('sha256').update(this.lint.content).digest('base64') + + this.meta = { + title: `Database Advisor: Lint ${this.lint.path}`, + } + + this.sections = [ + { + content: this.lint.content, + }, + ] + + return { checksum: this.checksum, meta: this.meta, sections: this.sections } + } + + extractIndexedContent(): string { + const sections = this.sections ?? [] + const sectionText = sections.map(({ content }) => content).join('\n\n') + return `# ${this.lint.path}\n\n${sectionText}` + } +} diff --git a/apps/studio/components/grid/components/menu/RowContextMenu.tsx b/apps/studio/components/grid/components/menu/RowContextMenu.tsx index dfa4d26bede..14a457eceb6 100644 --- a/apps/studio/components/grid/components/menu/RowContextMenu.tsx +++ b/apps/studio/components/grid/components/menu/RowContextMenu.tsx @@ -5,8 +5,9 @@ import { Item, ItemParams, Menu } from 'react-contexify' import type { SupaRow } from 'components/grid/types' import { useTableEditorStateSnapshot } from 'state/table-editor' import { useTableEditorTableStateSnapshot } from 'state/table-editor-table' +import { copyToClipboard } from 'ui' import { ROW_CONTEXT_MENU_ID } from '.' -import { copyToClipboard, formatClipboardValue } from '../../utils/common' +import { formatClipboardValue } from '../../utils/common' export type RowContextMenuProps = { rows: SupaRow[] diff --git a/apps/studio/components/grid/utils/common.ts b/apps/studio/components/grid/utils/common.ts index 781c9066c3e..b532c45cff3 100644 --- a/apps/studio/components/grid/utils/common.ts +++ b/apps/studio/components/grid/utils/common.ts @@ -5,12 +5,3 @@ export function formatClipboardValue(value: any) { } return value } - -export const copyToClipboard = (str: string, callback = () => {}) => { - const focused = window.document.hasFocus() - if (focused) { - window.navigator?.clipboard?.writeText(str).then(callback) - } else { - console.warn('Unable to copy to clipboard') - } -} diff --git a/apps/studio/components/interfaces/App/CommandMenu/ApiKeys.tsx b/apps/studio/components/interfaces/App/CommandMenu/ApiKeys.tsx index c86051bba3f..7010e1a8c82 100644 --- a/apps/studio/components/interfaces/App/CommandMenu/ApiKeys.tsx +++ b/apps/studio/components/interfaces/App/CommandMenu/ApiKeys.tsx @@ -3,8 +3,7 @@ import { useMemo } from 'react' import { getAPIKeys, useProjectSettingsV2Query } from 'data/config/project-settings-v2-query' import { useSelectedProject } from 'hooks/misc/useSelectedProject' -import { copyToClipboard } from 'lib/helpers' -import { Badge } from 'ui' +import { Badge, copyToClipboard } from 'ui' import type { ICommand } from 'ui-patterns/CommandMenu' import { PageType, diff --git a/apps/studio/components/interfaces/App/CommandMenu/ApiUrl.tsx b/apps/studio/components/interfaces/App/CommandMenu/ApiUrl.tsx index 50a6fb33c3b..dfad51d0667 100644 --- a/apps/studio/components/interfaces/App/CommandMenu/ApiUrl.tsx +++ b/apps/studio/components/interfaces/App/CommandMenu/ApiUrl.tsx @@ -2,8 +2,7 @@ import { Link } from 'lucide-react' import { useProjectSettingsV2Query } from 'data/config/project-settings-v2-query' import { useSelectedProject } from 'hooks/misc/useSelectedProject' -import { copyToClipboard } from 'lib/helpers' -import { Badge } from 'ui' +import { Badge, copyToClipboard } from 'ui' import { useRegisterCommands, useSetCommandMenuOpen } from 'ui-patterns/CommandMenu' import { COMMAND_MENU_SECTIONS } from './CommandMenu.utils' import { orderCommandSectionsByPriority } from './ordering' diff --git a/apps/studio/components/interfaces/App/RouteValidationWrapper.tsx b/apps/studio/components/interfaces/App/RouteValidationWrapper.tsx index ac381001c9f..f87aa4aa0c6 100644 --- a/apps/studio/components/interfaces/App/RouteValidationWrapper.tsx +++ b/apps/studio/components/interfaces/App/RouteValidationWrapper.tsx @@ -69,8 +69,8 @@ const RouteValidationWrapper = ({ children }: PropsWithChildren<{}>) => { const isValidOrg = organizations.some((org) => org.slug === slug) if (!isValidOrg) { - toast.error('This organization does not exist') - router.push(DEFAULT_HOME) + toast.error("We couldn't find that organization") + router.push(`${DEFAULT_HOME}?error=org_not_found&org=${slug}`) return } } diff --git a/apps/studio/components/interfaces/Auth/Hooks/HookCard.tsx b/apps/studio/components/interfaces/Auth/Hooks/HookCard.tsx index 80dc0264277..38bb4226a16 100644 --- a/apps/studio/components/interfaces/Auth/Hooks/HookCard.tsx +++ b/apps/studio/components/interfaces/Auth/Hooks/HookCard.tsx @@ -1,10 +1,9 @@ import { PermissionAction } from '@supabase/shared-types/out/constants' import { Check, Webhook } from 'lucide-react' -import { Badge, Input } from 'ui' +import { Badge, Input, copyToClipboard } from 'ui' import { ButtonTooltip } from 'components/ui/ButtonTooltip' import { useCheckPermissions } from 'hooks/misc/useCheckPermissions' -import { copyToClipboard } from 'lib/helpers' import { Hook } from './hooks.constants' interface HookCardProps { diff --git a/apps/studio/components/interfaces/Auth/Users/Users.utils.tsx b/apps/studio/components/interfaces/Auth/Users/Users.utils.tsx index 7f5be688a1c..ac1fee9298d 100644 --- a/apps/studio/components/interfaces/Auth/Users/Users.utils.tsx +++ b/apps/studio/components/interfaces/Auth/Users/Users.utils.tsx @@ -5,7 +5,6 @@ import { Column, useRowSelection } from 'react-data-grid' import { User } from 'data/auth/users-infinite-query' import { BASE_PATH } from 'lib/constants' -import { copyToClipboard } from 'lib/helpers' import { Checkbox_Shadcn_, cn, @@ -14,6 +13,7 @@ import { ContextMenuItem_Shadcn_, ContextMenuSeparator_Shadcn_, ContextMenuTrigger_Shadcn_, + copyToClipboard, } from 'ui' import { PROVIDERS_SCHEMAS } from '../AuthProvidersFormValidation' import { ColumnConfiguration, USERS_TABLE_COLUMNS } from './Users.constants' diff --git a/apps/studio/components/interfaces/Billing/Payment/PaymentConfirmation.tsx b/apps/studio/components/interfaces/Billing/Payment/PaymentConfirmation.tsx new file mode 100644 index 00000000000..5b568e36437 --- /dev/null +++ b/apps/studio/components/interfaces/Billing/Payment/PaymentConfirmation.tsx @@ -0,0 +1,39 @@ +import { useStripe } from '@stripe/react-stripe-js' +import { PaymentIntentResult } from '@stripe/stripe-js' +import { useEffect } from 'react' +import { LoadingLine } from 'ui' + +export const PaymentConfirmation = ({ + paymentIntentSecret, + onPaymentIntentConfirm, + onLoadingChange, + paymentMethodId, + onError, +}: { + paymentIntentSecret: string + paymentMethodId: string + onPaymentIntentConfirm: (response: PaymentIntentResult) => void + onLoadingChange: (loading: boolean) => void + onError?: (error: Error) => void +}) => { + const stripe = useStripe() + + useEffect(() => { + if (stripe && paymentIntentSecret) { + onLoadingChange(true) + stripe! + .confirmCardPayment(paymentIntentSecret, { payment_method: paymentMethodId }) + .then((res) => { + onPaymentIntentConfirm(res) + onLoadingChange(false) + }) + .catch((err) => { + console.error(err) + onError?.(err) + onLoadingChange(false) + }) + } + }, [paymentIntentSecret, stripe]) + + return +} diff --git a/apps/studio/components/interfaces/Connect/ConnectionParameters.tsx b/apps/studio/components/interfaces/Connect/ConnectionParameters.tsx index 3bb5f9edf62..d3654d9377c 100644 --- a/apps/studio/components/interfaces/Connect/ConnectionParameters.tsx +++ b/apps/studio/components/interfaces/Connect/ConnectionParameters.tsx @@ -1,13 +1,13 @@ import { Check, ChevronRight, Copy } from 'lucide-react' import { useState } from 'react' -import { copyToClipboard } from 'lib/helpers' import { Button, cn, Collapsible_Shadcn_, CollapsibleContent_Shadcn_, CollapsibleTrigger_Shadcn_, + copyToClipboard, Separator, } from 'ui' diff --git a/apps/studio/components/interfaces/Database/Tables/ColumnList.tsx b/apps/studio/components/interfaces/Database/Tables/ColumnList.tsx index cb595bc58cf..c77fee04b4e 100644 --- a/apps/studio/components/interfaces/Database/Tables/ColumnList.tsx +++ b/apps/studio/components/interfaces/Database/Tables/ColumnList.tsx @@ -1,6 +1,6 @@ import { PermissionAction } from '@supabase/shared-types/out/constants' import { noop } from 'lodash' -import { ChevronLeft, Edit, MoreVertical, Plus, Search, Trash } from 'lucide-react' +import { Check, ChevronLeft, Edit, MoreVertical, Plus, Search, Trash, X } from 'lucide-react' import Link from 'next/link' import { useState } from 'react' @@ -125,6 +125,9 @@ const ColumnList = ({ , Data Type, Format, + + Nullable + , , ]} body={columns.map((x) => ( @@ -145,6 +148,13 @@ const ColumnList = ({ {x.format} + + {x.is_nullable ? ( + + ) : ( + + )} + {!isLocked && isTableEntity && ( diff --git a/apps/studio/components/interfaces/Functions/CommandRender.tsx b/apps/studio/components/interfaces/Functions/CommandRender.tsx index 6b3a1afa803..8285ce8b3c2 100644 --- a/apps/studio/components/interfaces/Functions/CommandRender.tsx +++ b/apps/studio/components/interfaces/Functions/CommandRender.tsx @@ -1,6 +1,7 @@ import { Check, Clipboard } from 'lucide-react' import { forwardRef, useState } from 'react' -import { cn } from 'ui' + +import { cn, copyToClipboard } from 'ui' const CommandRender = forwardRef( ({ commands, className }, ref) => { @@ -35,10 +36,8 @@ const Command = ({ item }: any) => { onClick={() => { function onCopy(value: any) { setIsCopied(true) - navigator.clipboard.writeText(value).then() - setTimeout(function () { - setIsCopied(false) - }, 3000) + copyToClipboard(value) + setTimeout(() => setIsCopied(false), 3000) } onCopy(item.command) }} diff --git a/apps/studio/components/interfaces/Functions/EdgeFunctionsListItem.tsx b/apps/studio/components/interfaces/Functions/EdgeFunctionsListItem.tsx index a0951552654..0d99237296c 100644 --- a/apps/studio/components/interfaces/Functions/EdgeFunctionsListItem.tsx +++ b/apps/studio/components/interfaces/Functions/EdgeFunctionsListItem.tsx @@ -8,7 +8,7 @@ import { useProjectContext } from 'components/layouts/ProjectLayout/ProjectConte import Table from 'components/to-be-cleaned/Table' import { useCustomDomainsQuery } from 'data/custom-domains/custom-domains-query' import type { EdgeFunctionsResponse } from 'data/edge-functions/edge-functions-query' -import { Tooltip, TooltipContent, TooltipTrigger } from 'ui' +import { copyToClipboard, Tooltip, TooltipContent, TooltipTrigger } from 'ui' interface EdgeFunctionsListItemProps { function: EdgeFunctionsResponse @@ -55,10 +55,8 @@ export const EdgeFunctionsListItem = ({ function: item }: EdgeFunctionsListItemP onClick={(event: any) => { function onCopy(value: any) { setIsCopied(true) - navigator.clipboard.writeText(value).then() - setTimeout(function () { - setIsCopied(false) - }, 3000) + copyToClipboard(value) + setTimeout(() => setIsCopied(false), 3000) } event.stopPropagation() onCopy(endpoint) diff --git a/apps/studio/components/interfaces/Organization/BillingSettings/CreditTopUp.tsx b/apps/studio/components/interfaces/Organization/BillingSettings/CreditTopUp.tsx index eed17a4eb47..a4954c689e5 100644 --- a/apps/studio/components/interfaces/Organization/BillingSettings/CreditTopUp.tsx +++ b/apps/studio/components/interfaces/Organization/BillingSettings/CreditTopUp.tsx @@ -1,6 +1,6 @@ import HCaptcha from '@hcaptcha/react-hcaptcha' import { zodResolver } from '@hookform/resolvers/zod' -import { Elements, useStripe } from '@stripe/react-stripe-js' +import { Elements } from '@stripe/react-stripe-js' import { loadStripe, PaymentIntentResult } from '@stripe/stripe-js' import { PermissionAction, SupportCategories } from '@supabase/shared-types/out/constants' import { useQueryClient } from '@tanstack/react-query' @@ -35,10 +35,10 @@ import { Form_Shadcn_, FormField_Shadcn_, Input_Shadcn_, - LoadingLine, } from 'ui' import { FormItemLayout } from 'ui-patterns/form/FormItemLayout/FormItemLayout' import PaymentMethodSelection from './Subscription/PaymentMethodSelection' +import { PaymentConfirmation } from 'components/interfaces/Billing/Payment/PaymentConfirmation' const stripePromise = loadStripe(STRIPE_PUBLIC_KEY) @@ -331,34 +331,3 @@ export const CreditTopUp = ({ slug }: { slug: string | undefined }) => { ) } - -const PaymentConfirmation = ({ - paymentIntentSecret, - onPaymentIntentConfirm, - onLoadingChange, - paymentMethodId, -}: { - paymentIntentSecret: string - paymentMethodId: string - onPaymentIntentConfirm: (response: PaymentIntentResult) => void - onLoadingChange: (loading: boolean) => void -}) => { - const stripe = useStripe() - - useEffect(() => { - if (stripe && paymentIntentSecret) { - onLoadingChange(true) - stripe! - .confirmCardPayment(paymentIntentSecret, { payment_method: paymentMethodId }) - .then((res) => { - onPaymentIntentConfirm(res) - onLoadingChange(false) - }) - .catch((err) => { - onLoadingChange(false) - }) - } - }, [paymentIntentSecret, stripe]) - - return -} diff --git a/apps/studio/components/interfaces/Organization/BillingSettings/Subscription/PlanUpdateSidePanel.tsx b/apps/studio/components/interfaces/Organization/BillingSettings/Subscription/PlanUpdateSidePanel.tsx index cf47851064d..40f7fd83c70 100644 --- a/apps/studio/components/interfaces/Organization/BillingSettings/Subscription/PlanUpdateSidePanel.tsx +++ b/apps/studio/components/interfaces/Organization/BillingSettings/Subscription/PlanUpdateSidePanel.tsx @@ -27,6 +27,7 @@ import ExitSurveyModal from './ExitSurveyModal' import MembersExceedLimitModal from './MembersExceedLimitModal' import { SubscriptionPlanUpdateDialog } from './SubscriptionPlanUpdateDialog' import UpgradeSurveyModal from './UpgradeModal' +import PartnerManagedResource from 'components/ui/PartnerManagedResource' const PlanUpdateSidePanel = () => { const router = useRouter() @@ -143,6 +144,18 @@ const PlanUpdateSidePanel = () => { } > + {selectedOrganization?.managed_by === 'vercel-marketplace' && ( + + )}
{subscriptionsPlans.map((plan) => { @@ -208,6 +221,9 @@ const PlanUpdateSidePanel = () => { type={isDowngradeOption ? 'default' : 'primary'} disabled={ subscription?.plan?.id === 'enterprise' || + // Downgrades to free are still allowed through the dashboard given we have much better control about showing customers the impact + any possible issues with downgrading to free + (selectedOrganization?.managed_by !== 'supabase' && + plan.id !== 'tier_free') || hasOrioleProjects || !canUpdateSubscription } diff --git a/apps/studio/components/interfaces/Organization/GeneralSettings/DeleteOrganizationButton.tsx b/apps/studio/components/interfaces/Organization/GeneralSettings/DeleteOrganizationButton.tsx index ee52c8a84a4..d0c2d51407f 100644 --- a/apps/studio/components/interfaces/Organization/GeneralSettings/DeleteOrganizationButton.tsx +++ b/apps/studio/components/interfaces/Organization/GeneralSettings/DeleteOrganizationButton.tsx @@ -37,7 +37,7 @@ export const DeleteOrganizationButton = () => { if (!values.orgName) { errors.orgName = 'Enter the name of the organization.' } - if (values.orgName !== orgSlug) { + if (values.orgName.trim() !== orgSlug?.trim()) { errors.orgName = 'Value entered does not match the value above.' } return errors diff --git a/apps/studio/components/interfaces/Organization/NewOrg/NewOrgForm.tsx b/apps/studio/components/interfaces/Organization/NewOrg/NewOrgForm.tsx index 2f16d232697..8d52683c54e 100644 --- a/apps/studio/components/interfaces/Organization/NewOrg/NewOrgForm.tsx +++ b/apps/studio/components/interfaces/Organization/NewOrg/NewOrgForm.tsx @@ -1,11 +1,11 @@ -import { PaymentElement, useElements, useStripe } from '@stripe/react-stripe-js' -import type { PaymentMethod } from '@stripe/stripe-js' +import { Elements, PaymentElement, useElements, useStripe } from '@stripe/react-stripe-js' +import type { PaymentIntentResult, PaymentMethod, StripeElementsOptions } from '@stripe/stripe-js' import _ from 'lodash' -import { Edit2, ExternalLink, HelpCircle } from 'lucide-react' +import { ExternalLink, HelpCircle } from 'lucide-react' import Link from 'next/link' import { useRouter } from 'next/router' import { parseAsString, useQueryStates } from 'nuqs' -import { useEffect, useMemo, useState } from 'react' +import { forwardRef, useEffect, useImperativeHandle, useMemo, useRef, useState } from 'react' import { toast } from 'sonner' import { z } from 'zod' @@ -16,9 +16,7 @@ import { useOrganizationCreateMutation } from 'data/organizations/organization-c import { useOrganizationsQuery } from 'data/organizations/organizations-query' import { useProjectsQuery } from 'data/projects/projects-query' import { useLocalStorageQuery } from 'hooks/misc/useLocalStorage' -import { BASE_PATH, PRICING_TIER_LABELS_ORG } from 'lib/constants' -import { getURL } from 'lib/helpers' -import { useProfile } from 'lib/profile' +import { PRICING_TIER_LABELS_ORG, STRIPE_PUBLIC_KEY } from 'lib/constants' import { Button, Input, @@ -37,6 +35,13 @@ import { import ConfirmationModal from 'ui-patterns/Dialogs/ConfirmationModal' import { BillingCustomerDataNewOrgDialog } from '../BillingSettings/BillingCustomerData/BillingCustomerDataNewOrgDialog' import { FormCustomerData } from '../BillingSettings/BillingCustomerData/useBillingCustomerDataForm' +import { useConfirmPendingSubscriptionChangeMutation } from 'data/subscriptions/org-subscription-confirm-pending-change' +import { loadStripe } from '@stripe/stripe-js' +import { useTheme } from 'next-themes' +import { SetupIntentResponse } from 'data/stripe/setup-intent-mutation' +import { useProfile } from 'lib/profile' +import { PaymentConfirmation } from 'components/interfaces/Billing/Payment/PaymentConfirmation' +import { getURL } from 'lib/helpers' const ORG_KIND_TYPES = { PERSONAL: 'Personal', @@ -59,6 +64,8 @@ const ORG_SIZE_DEFAULT = '1' interface NewOrgFormProps { onPaymentMethodReset: () => void + setupIntent?: SetupIntentResponse + onPlanSelected: (plan: string) => void } const formSchema = z.object({ @@ -79,17 +86,18 @@ const formSchema = z.object({ type FormState = z.infer +const stripePromise = loadStripe(STRIPE_PUBLIC_KEY) + /** * No org selected yet, create a new one * [Joshen] Need to refactor to use Form_Shadcn here */ -const NewOrgForm = ({ onPaymentMethodReset }: NewOrgFormProps) => { +const NewOrgForm = ({ onPaymentMethodReset, setupIntent, onPlanSelected }: NewOrgFormProps) => { const router = useRouter() const user = useProfile() const { data: organizations, isSuccess } = useOrganizationsQuery() const { data: projects } = useProjectsQuery() - const stripe = useStripe() - const elements = useElements() + const { resolvedTheme } = useTheme() const [lastVisitedOrganization] = useLocalStorageQuery( LOCAL_STORAGE_KEYS.LAST_VISITED_ORGANIZATION, @@ -107,6 +115,21 @@ const NewOrgForm = ({ onPaymentMethodReset }: NewOrgFormProps) => { const [customerData, setCustomerData] = useState(null) + const stripeOptionsPaymentMethod: StripeElementsOptions = useMemo( + () => + ({ + clientSecret: setupIntent ? setupIntent.client_secret! : '', + appearance: { + theme: resolvedTheme?.includes('dark') ? 'night' : 'flat', + labels: 'floating', + }, + ...(setupIntent?.pending_subscription_flow_enabled_for_creation === true + ? { paymentMethodCreation: 'manual' } + : {}), + }) as const, + [setupIntent, resolvedTheme] + ) + const [formState, setFormState] = useState({ plan: 'FREE', name: '', @@ -118,7 +141,6 @@ const NewOrgForm = ({ onPaymentMethodReset }: NewOrgFormProps) => { const [searchParams] = useQueryStates({ returnTo: parseAsString.withDefault(''), auth_id: parseAsString.withDefault(''), - token: parseAsString.withDefault(''), }) const updateForm = (key: keyof FormState, value: unknown) => { @@ -132,7 +154,10 @@ const NewOrgForm = ({ onPaymentMethodReset }: NewOrgFormProps) => { if (typeof name === 'string') updateForm('name', name) if (typeof kind === 'string') updateForm('kind', kind) - if (typeof plan === 'string') updateForm('plan', plan) + if (typeof plan === 'string') { + updateForm('plan', plan) + onPlanSelected(plan) + } if (typeof size === 'string') updateForm('size', size) if (typeof spend_cap === 'string') updateForm('spend_cap', spend_cap === 'true') }, [router.isReady]) @@ -147,26 +172,16 @@ const NewOrgForm = ({ onPaymentMethodReset }: NewOrgFormProps) => { const [newOrgLoading, setNewOrgLoading] = useState(false) const [paymentMethod, setPaymentMethod] = useState() + const [paymentConfirmationLoading, setPaymentConfirmationLoading] = useState(false) const [showSpendCapHelperModal, setShowSpendCapHelperModal] = useState(false) + const [paymentIntentSecret, setPaymentIntentSecret] = useState(null) const { mutate: createOrganization } = useOrganizationCreateMutation({ onSuccess: async (org) => { - const prefilledProjectName = user.profile?.username - ? user.profile.username + `'s Project` - : 'My Project' - - if (searchParams.returnTo && searchParams.auth_id) { - router.push( - `${searchParams.returnTo}?auth_id=${searchParams.auth_id}${ - searchParams.token && `&token=${searchParams.token}&org=${org.name}` - }`, - undefined, - { - shallow: false, - } - ) + if ('pending_payment_intent_secret' in org && org.pending_payment_intent_secret) { + setPaymentIntentSecret(org.pending_payment_intent_secret) } else { - router.push(`/new/${org.slug}?projectName=${prefilledProjectName}`) + onOrganizationCreated(org as { slug: string }) } }, onError: (data) => { @@ -176,11 +191,59 @@ const NewOrgForm = ({ onPaymentMethodReset }: NewOrgFormProps) => { }, }) + const { mutate: confirmPendingSubscriptionChange } = useConfirmPendingSubscriptionChangeMutation({ + onSuccess: (data) => { + if (data && 'slug' in data) { + onOrganizationCreated({ slug: data.slug }) + } + }, + }) + + const paymentIntentConfirmed = async (paymentIntentConfirmation: PaymentIntentResult) => { + // Reset payment intent secret to ensure another attempt works as expected + setPaymentIntentSecret('') + + if (paymentIntentConfirmation.paymentIntent?.status === 'succeeded') { + await confirmPendingSubscriptionChange({ + payment_intent_id: paymentIntentConfirmation.paymentIntent.id, + name: formState.name, + kind: formState.kind, + size: formState.size, + }) + } else { + // If the payment intent is not successful, we reset the payment method and show an error + toast.error(`Could not confirm payment. Please try again or use a different card.`) + resetPaymentMethod() + setNewOrgLoading(false) + } + } + + const onOrganizationCreated = (org: { slug: string }) => { + const prefilledProjectName = user.profile?.username + ? user.profile.username + `'s Project` + : 'My Project' + + if (searchParams.returnTo && searchParams.auth_id) { + router.push(`${searchParams.returnTo}?auth_id=${searchParams.auth_id}`, undefined, { + shallow: false, + }) + } else { + router.push(`/new/${org.slug}?projectName=${prefilledProjectName}`) + } + } + function validateOrgName(name: any) { const value = name ? name.trim() : '' return value.length >= 1 } + const stripeOptionsConfirm = useMemo(() => { + return { + clientSecret: paymentIntentSecret, + appearance: { theme: resolvedTheme?.includes('dark') ? 'night' : 'flat', labels: 'floating' }, + } as StripeElementsOptions + }, [paymentIntentSecret, resolvedTheme]) + async function createOrg(paymentMethodId?: string) { const dbTier = formState.plan === 'PRO' && !formState.spend_cap ? 'PAYG' : formState.plan @@ -201,35 +264,21 @@ const NewOrgForm = ({ onPaymentMethodReset }: NewOrgFormProps) => { }) } - const handleSubmit = async () => { - if (!stripe || !elements) { - return console.error('Stripe.js has not loaded') - } + const paymentRef = useRef<{ createPaymentMethod: () => Promise }>(null) + const handleSubmit = async () => { setNewOrgLoading(true) if (formState.plan === 'FREE') { await createOrg() } else if (!paymentMethod) { - const { error, setupIntent } = await stripe.confirmSetup({ - elements, - redirect: 'if_required', - confirmParams: { - return_url: `${getURL()}/new`, - expand: ['payment_method'], - }, - }) - - if (error || !setupIntent.payment_method) { - toast.error(error?.message ?? ' Failed to save card details') + const paymentMethod = await paymentRef.current?.createPaymentMethod() + if (paymentMethod) { + setPaymentMethod(paymentMethod) + createOrg(paymentMethod.id) + } else { setNewOrgLoading(false) - return } - - const paymentMethodFromSetup = setupIntent.payment_method as PaymentMethod - - setPaymentMethod(paymentMethodFromSetup) - createOrg(paymentMethodFromSetup.id) } else { createOrg(paymentMethod.id) } @@ -269,7 +318,7 @@ const NewOrgForm = ({ onPaymentMethodReset }: NewOrgFormProps) => {
-
- ) : ( - - )} + + + )} @@ -606,8 +633,88 @@ const NewOrgForm = ({ onPaymentMethodReset }: NewOrgFormProps) => { })} + + {stripePromise && paymentIntentSecret && paymentMethod && ( + + + paymentIntentConfirmed(paymentIntentConfirmation) + } + onLoadingChange={(loading) => setPaymentConfirmationLoading(loading)} + paymentMethodId={paymentMethod.id} + onError={(err) => { + toast.error(`Failed to confirm payment: ${err.message}`) + setNewOrgLoading(false) + resetPaymentMethod() + }} + /> + + )} ) } export default NewOrgForm + +/** + * Set up as a separate component, as we need any component using stripe/elements to be wrapped in Elements. + * + * If Elements is on a higher level, we risk losing all form state in case a payment fails. + */ +const Payment = forwardRef( + ( + { + pending_subscription_flow_enabled_for_creation, + }: { pending_subscription_flow_enabled_for_creation: boolean }, + ref + ) => { + const stripe = useStripe() + const elements = useElements() + + const createPaymentMethod = async () => { + if (!stripe || !elements) return + await elements.submit() + + if (pending_subscription_flow_enabled_for_creation) { + // To avoid double 3DS confirmation, we just create the payment method here, as there might be a confirmation step while doing the actual payment + const { error, paymentMethod } = await stripe.createPaymentMethod({ + elements, + }) + if (error || paymentMethod == null) { + toast.error(error?.message ?? ' Failed to process card details') + return + } + return paymentMethod + } else { + const { error, setupIntent } = await stripe.confirmSetup({ + elements, + redirect: 'if_required', + confirmParams: { + return_url: `${getURL()}/new`, + expand: ['payment_method'], + }, + }) + + if (error || !setupIntent.payment_method) { + toast.error(error?.message ?? ' Failed to save card details') + return + } + + return setupIntent.payment_method as PaymentMethod + } + } + + useImperativeHandle(ref, () => ({ + createPaymentMethod, + })) + + return ( + + + + ) + } +) + +Payment.displayName = 'Payment' diff --git a/apps/studio/components/interfaces/ProjectAPIDocs/Content/Introduction.tsx b/apps/studio/components/interfaces/ProjectAPIDocs/Content/Introduction.tsx index 10edf7179ef..9ec3c001396 100644 --- a/apps/studio/components/interfaces/ProjectAPIDocs/Content/Introduction.tsx +++ b/apps/studio/components/interfaces/ProjectAPIDocs/Content/Introduction.tsx @@ -1,8 +1,7 @@ import { useParams } from 'common' -import { Button, Input } from 'ui' +import { Button, Input, copyToClipboard } from 'ui' import { useProjectSettingsV2Query } from 'data/config/project-settings-v2-query' -import { copyToClipboard } from 'lib/helpers' import { Copy } from 'lucide-react' import { useEffect, useState } from 'react' import ContentSnippet from '../ContentSnippet' diff --git a/apps/studio/components/interfaces/SQLEditor/SQLEditor.tsx b/apps/studio/components/interfaces/SQLEditor/SQLEditor.tsx index 4d86bfb86e1..054174f05f3 100644 --- a/apps/studio/components/interfaces/SQLEditor/SQLEditor.tsx +++ b/apps/studio/components/interfaces/SQLEditor/SQLEditor.tsx @@ -316,6 +316,7 @@ export const SQLEditor = () => { sql: wrapWithRoleImpersonation(formattedSql, impersonatedRoleState), autoLimit: appendAutoLimit ? limit : undefined, isRoleImpersonationEnabled: isRoleImpersonationEnabled(impersonatedRoleState.role), + isStatementTimeoutDisabled: true, contextualInvalidation: true, handleError: (error) => { throw error diff --git a/apps/studio/components/interfaces/SQLEditor/UtilityPanel/Results.tsx b/apps/studio/components/interfaces/SQLEditor/UtilityPanel/Results.tsx index a4bc9080273..32a96d76107 100644 --- a/apps/studio/components/interfaces/SQLEditor/UtilityPanel/Results.tsx +++ b/apps/studio/components/interfaces/SQLEditor/UtilityPanel/Results.tsx @@ -3,13 +3,13 @@ import { useState } from 'react' import DataGrid, { CalculatedColumn } from 'react-data-grid' import { handleCopyCell } from 'components/grid/SupabaseGrid.utils' -import { copyToClipboard } from 'lib/helpers' import { cn, ContextMenu_Shadcn_, ContextMenuContent_Shadcn_, ContextMenuItem_Shadcn_, ContextMenuTrigger_Shadcn_, + copyToClipboard, } from 'ui' import { CellDetailPanel } from './CellDetailPanel' diff --git a/apps/studio/components/interfaces/Settings/Logs/LogTable.tsx b/apps/studio/components/interfaces/Settings/Logs/LogTable.tsx index c19f5dde1b0..2006ee3dd82 100644 --- a/apps/studio/components/interfaces/Settings/Logs/LogTable.tsx +++ b/apps/studio/components/interfaces/Settings/Logs/LogTable.tsx @@ -11,10 +11,16 @@ import { ButtonTooltip } from 'components/ui/ButtonTooltip' import { DownloadResultsButton } from 'components/ui/DownloadResultsButton' import { useSelectedLog } from 'hooks/analytics/useSelectedLog' import { useCheckPermissions } from 'hooks/misc/useCheckPermissions' -import { copyToClipboard } from 'lib/helpers' import { useProfile } from 'lib/profile' import { ResponseError } from 'types' -import { Button, ResizableHandle, ResizablePanel, ResizablePanelGroup, cn } from 'ui' +import { + Button, + ResizableHandle, + ResizablePanel, + ResizablePanelGroup, + cn, + copyToClipboard, +} from 'ui' import AuthColumnRenderer from './LogColumnRenderers/AuthColumnRenderer' import DatabaseApiColumnRender from './LogColumnRenderers/DatabaseApiColumnRender' import DatabasePostgresColumnRender from './LogColumnRenderers/DatabasePostgresColumnRender' diff --git a/apps/studio/components/interfaces/Settings/Logs/Logs.DatePickers.tsx b/apps/studio/components/interfaces/Settings/Logs/Logs.DatePickers.tsx index a113cd1539f..f3576372338 100644 --- a/apps/studio/components/interfaces/Settings/Logs/Logs.DatePickers.tsx +++ b/apps/studio/components/interfaces/Settings/Logs/Logs.DatePickers.tsx @@ -7,10 +7,16 @@ import DatePicker from 'react-datepicker' import { Label } from '@ui/components/shadcn/ui/label' import { RadioGroup, RadioGroupItem } from '@ui/components/shadcn/ui/radio-group' import TimeSplitInput from 'components/ui/DatePicker/TimeSplitInput' -import { Button, PopoverContent_Shadcn_, PopoverTrigger_Shadcn_, Popover_Shadcn_, cn } from 'ui' +import { + Button, + PopoverContent_Shadcn_, + PopoverTrigger_Shadcn_, + Popover_Shadcn_, + cn, + copyToClipboard, +} from 'ui' import { LOGS_LARGE_DATE_RANGE_DAYS_THRESHOLD } from './Logs.constants' import type { DatetimeHelper } from './Logs.types' -import { copyToClipboard } from 'lib/helpers' export type DatePickerValue = { to: string diff --git a/apps/studio/components/interfaces/Settings/Logs/LogsQueryPanel.tsx b/apps/studio/components/interfaces/Settings/Logs/LogsQueryPanel.tsx index d6bebe2d1db..5ea8b4292a3 100644 --- a/apps/studio/components/interfaces/Settings/Logs/LogsQueryPanel.tsx +++ b/apps/studio/components/interfaces/Settings/Logs/LogsQueryPanel.tsx @@ -1,17 +1,17 @@ +import dayjs from 'dayjs' +import { BookOpen, Check, ChevronDown, Clipboard, ExternalLink, X } from 'lucide-react' import Link from 'next/link' import { ReactNode, useState } from 'react' import { IS_PLATFORM } from 'common' import Table from 'components/to-be-cleaned/Table' -import dayjs from 'dayjs' import { useIsFeatureEnabled } from 'hooks/misc/useIsFeatureEnabled' import { useFlag } from 'hooks/ui/useFlag' -import { copyToClipboard } from 'lib/helpers' -import { BookOpen, Check, ChevronDown, Clipboard, ExternalLink, X } from 'lucide-react' import { logConstants } from 'shared-data' import { Badge, Button, + copyToClipboard, DropdownMenu, DropdownMenuContent, DropdownMenuItem, diff --git a/apps/studio/components/layouts/TableEditorLayout/EntityListItem.tsx b/apps/studio/components/layouts/TableEditorLayout/EntityListItem.tsx index 6387d26cb6a..d1e8269143b 100644 --- a/apps/studio/components/layouts/TableEditorLayout/EntityListItem.tsx +++ b/apps/studio/components/layouts/TableEditorLayout/EntityListItem.tsx @@ -27,11 +27,11 @@ import { isTableLike } from 'data/table-editor/table-editor-types' import { fetchAllTableRows } from 'data/table-rows/table-rows-query' import { useQuerySchemaState } from 'hooks/misc/useSchemaQueryState' import { formatSql } from 'lib/formatSql' -import { copyToClipboard } from 'lib/helpers' import { useTableEditorStateSnapshot } from 'state/table-editor' import { createTabId, useTabsStateSnapshot } from 'state/tabs' import { cn, + copyToClipboard, DropdownMenu, DropdownMenuContent, DropdownMenuItem, diff --git a/apps/studio/components/to-be-cleaned/Storage/StorageExplorer/StorageExplorer.utils.tsx b/apps/studio/components/to-be-cleaned/Storage/StorageExplorer/StorageExplorer.utils.tsx index f2f2d01d609..e20a143b53b 100644 --- a/apps/studio/components/to-be-cleaned/Storage/StorageExplorer/StorageExplorer.utils.tsx +++ b/apps/studio/components/to-be-cleaned/Storage/StorageExplorer/StorageExplorer.utils.tsx @@ -1,8 +1,8 @@ +import { toast } from 'sonner' + import { downloadBucketObject } from 'data/storage/bucket-object-download-mutation' import { StorageObject } from 'data/storage/bucket-objects-list-mutation' -import { copyToClipboard } from 'lib/helpers' -import { toast } from 'sonner' -import { SONNER_DEFAULT_DURATION } from 'ui' +import { SONNER_DEFAULT_DURATION, copyToClipboard } from 'ui' import { STORAGE_ROW_STATUS, STORAGE_ROW_TYPES } from '../Storage.constants' import { StorageItem, StorageItemMetadata, StorageItemWithColumn } from '../Storage.types' diff --git a/apps/studio/components/to-be-cleaned/Storage/StorageExplorer/useCopyUrl.tsx b/apps/studio/components/to-be-cleaned/Storage/StorageExplorer/useCopyUrl.tsx index e31ae431a0f..44a9f3b8125 100644 --- a/apps/studio/components/to-be-cleaned/Storage/StorageExplorer/useCopyUrl.tsx +++ b/apps/studio/components/to-be-cleaned/Storage/StorageExplorer/useCopyUrl.tsx @@ -3,8 +3,8 @@ import { toast } from 'sonner' import { useProjectSettingsV2Query } from 'data/config/project-settings-v2-query' import { useCustomDomainsQuery } from 'data/custom-domains/custom-domains-query' -import { copyToClipboard } from 'lib/helpers' import { useStorageExplorerStateSnapshot } from 'state/storage-explorer' +import { copyToClipboard } from 'ui' import { URL_EXPIRY_DURATION } from '../Storage.constants' import { fetchFileUrl } from './useFetchFileUrlQuery' diff --git a/apps/studio/components/ui/CopyButton.tsx b/apps/studio/components/ui/CopyButton.tsx index 38e2848a9f3..9ac56ba9df6 100644 --- a/apps/studio/components/ui/CopyButton.tsx +++ b/apps/studio/components/ui/CopyButton.tsx @@ -1,8 +1,7 @@ import { Check, Clipboard } from 'lucide-react' -import React, { forwardRef, useEffect, useState } from 'react' +import { ComponentProps, forwardRef, useEffect, useState } from 'react' -import { copyToClipboard } from 'lib/helpers' -import { Button, cn } from 'ui' +import { Button, cn, copyToClipboard } from 'ui' type CopyButtonBaseProps = { iconOnly?: boolean @@ -22,7 +21,7 @@ type CopyButtonWithAsyncText = CopyButtonBaseProps & { } export type CopyButtonProps = (CopyButtonWithText | CopyButtonWithAsyncText) & - React.ComponentProps + ComponentProps const CopyButton = forwardRef( ( @@ -52,7 +51,7 @@ const CopyButton = forwardRef( onClick={async (e) => { const textToCopy = asyncText ? await asyncText() : text setShowCopied(true) - await copyToClipboard(textToCopy!) + copyToClipboard(textToCopy) onClick?.(e) }} {...props} diff --git a/apps/studio/components/ui/DownloadResultsButton.tsx b/apps/studio/components/ui/DownloadResultsButton.tsx index ba2dee032bd..715e5aa7acb 100644 --- a/apps/studio/components/ui/DownloadResultsButton.tsx +++ b/apps/studio/components/ui/DownloadResultsButton.tsx @@ -5,9 +5,9 @@ import Papa from 'papaparse' import { useMemo } from 'react' import { toast } from 'sonner' -import { copyToClipboard } from 'lib/helpers' import { Button, + copyToClipboard, DropdownMenu, DropdownMenuContent, DropdownMenuItem, diff --git a/apps/studio/components/ui/Logs/LogsExplorerHeader.tsx b/apps/studio/components/ui/Logs/LogsExplorerHeader.tsx index 547924858ab..ae32da4ba62 100644 --- a/apps/studio/components/ui/Logs/LogsExplorerHeader.tsx +++ b/apps/studio/components/ui/Logs/LogsExplorerHeader.tsx @@ -4,9 +4,16 @@ import { useState } from 'react' import { LOGS_EXPLORER_DOCS_URL } from 'components/interfaces/Settings/Logs/Logs.constants' import Table from 'components/to-be-cleaned/Table' -import { copyToClipboard } from 'lib/helpers' import { logConstants } from 'shared-data' -import { Button, SidePanel, Tabs, Tooltip, TooltipContent, TooltipTrigger } from 'ui' +import { + Button, + SidePanel, + Tabs, + Tooltip, + TooltipContent, + TooltipTrigger, + copyToClipboard, +} from 'ui' import { DocsButton } from '../DocsButton' export interface LogsExplorerHeaderProps { diff --git a/apps/studio/data/organizations/organization-create-mutation.ts b/apps/studio/data/organizations/organization-create-mutation.ts index 4273e98b2d3..3c28c0a4941 100644 --- a/apps/studio/data/organizations/organization-create-mutation.ts +++ b/apps/studio/data/organizations/organization-create-mutation.ts @@ -62,21 +62,24 @@ export const useOrganizationCreateMutation = ({ (vars) => createOrganization(vars), { async onSuccess(data, variables, context) { - // [Joshen] We're manually updating the query client here as the org's subscription is - // created async, and the invalidation will happen too quick where the GET organizations - // endpoint will error out with a 500 since the subscription isn't created yet. - queryClient.setQueriesData( - { - queryKey: organizationKeys.list(), - exact: true, - }, - (prev: any) => { - if (!prev) return prev - return [...prev, castOrganizationResponseToOrganization(data)] - } - ) + if (data && !('pending_payment_intent_secret' in data)) { + // [Joshen] We're manually updating the query client here as the org's subscription is + // created async, and the invalidation will happen too quick where the GET organizations + // endpoint will error out with a 500 since the subscription isn't created yet. + queryClient.setQueriesData( + { + queryKey: organizationKeys.list(), + exact: true, + }, + (prev: any) => { + if (!prev) return prev + return [...prev, castOrganizationResponseToOrganization(data)] + } + ) + + await queryClient.invalidateQueries(permissionKeys.list()) + } - await queryClient.invalidateQueries(permissionKeys.list()) await onSuccess?.(data, variables, context) }, async onError(data, variables, context) { diff --git a/apps/studio/data/organizations/organization-project-claim-mutation.ts b/apps/studio/data/organizations/organization-project-claim-mutation.ts index 16d42a31571..25b29c20ea4 100644 --- a/apps/studio/data/organizations/organization-project-claim-mutation.ts +++ b/apps/studio/data/organizations/organization-project-claim-mutation.ts @@ -10,6 +10,7 @@ type OrganizationProjectClaimVariables = { } async function claimOrganizationProject({ slug, token }: OrganizationProjectClaimVariables) { + // @ts-expect-error the endpoint is hidden in the spec for now const { data, error } = await post('/v1/organizations/{slug}/project-claim/{token}', { params: { path: { slug, token } }, }) diff --git a/apps/studio/data/organizations/organization-project-claim-query.ts b/apps/studio/data/organizations/organization-project-claim-query.ts index b02534661a2..f620074a041 100644 --- a/apps/studio/data/organizations/organization-project-claim-query.ts +++ b/apps/studio/data/organizations/organization-project-claim-query.ts @@ -1,7 +1,5 @@ import { useQuery, UseQueryOptions } from '@tanstack/react-query' import { organizationKeys } from './keys' - -import { components } from 'api-types' import { get, handleError } from 'data/fetchers' import type { ResponseError } from 'types' @@ -10,8 +8,42 @@ type OrganizationProjectClaimVariables = { token: string } -export type OrganizationProjectClaimResponse = - components['schemas']['OrganizationProjectClaimResponse'] +// todo replace with API type once exposed again - components['schemas']['OrganizationProjectClaimResponse'] +export type OrganizationProjectClaimResponse = { + created_at: string + /** Format: uuid */ + created_by: string + expires_at: string + preview: { + errors: { + key: string + message: string + }[] + info: { + key: string + message: string + }[] + members_exceeding_free_project_limit: { + limit: number + name: string + }[] + /** @enum {string} */ + source_subscription_plan: 'free' | 'pro' | 'team' | 'enterprise' + target_organization_eligible: boolean | null + target_organization_has_free_project_slots: boolean | null + /** @enum {string|null} */ + target_subscription_plan: 'free' | 'pro' | 'team' | 'enterprise' | null + valid: boolean + warnings: { + key: string + message: string + }[] + } + project: { + name: string + ref: string + } +} async function getOrganizationProjectClaim( { slug, token }: OrganizationProjectClaimVariables, @@ -19,13 +51,14 @@ async function getOrganizationProjectClaim( ) { if (!slug || !token) throw new Error('Slug and token are required') + // @ts-expect-error the endpoint is hidden in the spec for now const { data, error } = await get(`/v1/organizations/{slug}/project-claim/{token}`, { params: { path: { slug, token } }, signal, }) if (error) handleError(error) - return data + return data as OrganizationProjectClaimResponse } export type OrganizationProjectClaimData = Awaited> diff --git a/apps/studio/data/sql/execute-sql-query.ts b/apps/studio/data/sql/execute-sql-query.ts index 8e2b1536721..a7fe145fc9e 100644 --- a/apps/studio/data/sql/execute-sql-query.ts +++ b/apps/studio/data/sql/execute-sql-query.ts @@ -17,6 +17,7 @@ export type ExecuteSqlVariables = { queryKey?: QueryKey handleError?: (error: ResponseError) => { result: any } isRoleImpersonationEnabled?: boolean + isStatementTimeoutDisabled?: boolean autoLimit?: number contextualInvalidation?: boolean } @@ -29,6 +30,7 @@ export async function executeSql( queryKey, handleError, isRoleImpersonationEnabled = false, + isStatementTimeoutDisabled = false, }: Pick< ExecuteSqlVariables, | 'projectRef' @@ -37,6 +39,7 @@ export async function executeSql( | 'queryKey' | 'handleError' | 'isRoleImpersonationEnabled' + | 'isStatementTimeoutDisabled' >, signal?: AbortSignal, headersInit?: HeadersInit, @@ -80,7 +83,7 @@ export async function executeSql( .join('-') ?? '', }, }, - body: { query: sql }, + body: { query: sql, disable_statement_timeout: isStatementTimeoutDisabled }, headers, }) diff --git a/apps/studio/data/subscriptions/org-subscription-confirm-pending-change.ts b/apps/studio/data/subscriptions/org-subscription-confirm-pending-change.ts new file mode 100644 index 00000000000..ed6065771c8 --- /dev/null +++ b/apps/studio/data/subscriptions/org-subscription-confirm-pending-change.ts @@ -0,0 +1,92 @@ +import { useMutation, UseMutationOptions, useQueryClient } from '@tanstack/react-query' +import { toast } from 'sonner' + +import { handleError, post } from 'data/fetchers' +import type { ResponseError } from 'types' +import { organizationKeys } from 'data/organizations/keys' +import { permissionKeys } from 'data/permissions/keys' +import { castOrganizationResponseToOrganization } from 'data/organizations/organizations-query' +import type { components } from 'api-types' + +export type PendingSubscriptionChangeVariables = { + payment_intent_id: string + name: string + kind?: string + size?: string +} + +export async function confirmPendingSubscriptionChange({ + payment_intent_id, + name, + kind, + size, +}: PendingSubscriptionChangeVariables) { + const { data, error } = await post('/platform/organizations/confirm-subscription', { + body: { + payment_intent_id, + name, + kind, + size, + }, + }) + + if (error) handleError(error) + return data +} + +type PendingSubscriptionChangeData = Awaited> + +export const useConfirmPendingSubscriptionChangeMutation = ({ + onSuccess, + onError, + ...options +}: Omit< + UseMutationOptions< + PendingSubscriptionChangeData, + ResponseError, + PendingSubscriptionChangeVariables + >, + 'mutationFn' +> = {}) => { + const queryClient = useQueryClient() + + return useMutation< + PendingSubscriptionChangeData, + ResponseError, + PendingSubscriptionChangeVariables + >((vars) => confirmPendingSubscriptionChange(vars), { + async onSuccess(data, variables, context) { + // [Joshen] We're manually updating the query client here as the org's subscription is + // created async, and the invalidation will happen too quick where the GET organizations + // endpoint will error out with a 500 since the subscription isn't created yet. + queryClient.setQueriesData( + { + queryKey: organizationKeys.list(), + exact: true, + }, + (prev: any) => { + if (!prev) return prev + return [ + ...prev, + castOrganizationResponseToOrganization( + data as components['schemas']['OrganizationResponse'] + ), + ] + } + ) + + await queryClient.invalidateQueries(permissionKeys.list()) + + // todo replace plan in org + await onSuccess?.(data, variables, context) + }, + async onError(data, variables, context) { + if (onError === undefined) { + toast.error(`Failed to confirm payment: ${data.message}`) + } else { + onError(data, variables, context) + } + }, + ...options, + }) +} diff --git a/apps/studio/lib/helpers.test.ts b/apps/studio/lib/helpers.test.ts index 63e4f88ebcf..419e90a75eb 100644 --- a/apps/studio/lib/helpers.test.ts +++ b/apps/studio/lib/helpers.test.ts @@ -1,29 +1,30 @@ -import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest' +import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest' import { - tryParseJson, - minifyJSON, - prettifyJSON, - removeJSONTrailingComma, - timeout, - getURL, - makeRandomString, - pluckObjectFields, - tryParseInt, - propsAreEqual, - formatBytes, - snakeToCamel, - copyToClipboard, detectBrowser, detectOS, - pluralize, - isValidHttpUrl, - removeCommentsFromSql, - getSemanticVersion, + formatBytes, + formatCurrency, getDatabaseMajorVersion, getDistanceLatLonKM, - formatCurrency, + getSemanticVersion, + getURL, + isValidHttpUrl, + makeRandomString, + minifyJSON, + pluckObjectFields, + pluralize, + prettifyJSON, + propsAreEqual, + removeCommentsFromSql, + removeJSONTrailingComma, + snakeToCamel, + timeout, + tryParseInt, + tryParseJson, } from './helpers' +import { copyToClipboard } from 'ui' + describe('tryParseJson', () => { it('should return the parsed JSON', () => { const result = tryParseJson('{"test": "test"}') @@ -170,6 +171,9 @@ describe('copyToClipboard', () => { }, }) + // CopyToClipboard uses setTimeout to call the callback + vi.useFakeTimers() + // If ClipboardItem is used vi.stubGlobal('ClipboardItem', function (items: any) { return items @@ -181,10 +185,12 @@ describe('copyToClipboard', () => { afterEach(() => { vi.unstubAllGlobals() + vi.useRealTimers() }) it('uses clipboard.write if available', async () => { await copyToClipboard('hello') + vi.runAllTimers() expect(writeMock).toHaveBeenCalled() }) diff --git a/apps/studio/lib/helpers.ts b/apps/studio/lib/helpers.ts index e5baf72bfd8..268297a597e 100644 --- a/apps/studio/lib/helpers.ts +++ b/apps/studio/lib/helpers.ts @@ -1,6 +1,3 @@ -import { noop } from 'lodash' -import { toast } from 'sonner' - export { default as passwordStrength } from './password-strength' export { default as uuidv4 } from './uuid' @@ -146,42 +143,6 @@ export const snakeToCamel = (str: string) => group.toUpperCase().replace('-', '').replace('_', '') ) -/** - * Copy text content (string or Promise) into Clipboard. Safari doesn't support write text into clipboard async, - * so if you need to load text content async before coping, please use Promise for the 1st arg. - * - * IF YOU NEED TO CHANGE THIS FUNCTION, PLEASE TEST IT IN SAFARI with a promised string. Expiring URL to a file in a - * private bucket will do. - * - * Copied code from https://wolfgangrittner.dev/how-to-use-clipboard-api-in-firefox/ - */ -export const copyToClipboard = async (str: string | Promise, callback = noop) => { - const focused = window.document.hasFocus() - if (focused) { - if (typeof ClipboardItem && navigator.clipboard?.write) { - // NOTE: Safari locks down the clipboard API to only work when triggered - // by a direct user interaction. You can't use it async in a promise. - // But! You can wrap the promise in a ClipboardItem, and give that to - // the clipboard API. - // Found this on https://developer.apple.com/forums/thread/691873 - const text = new ClipboardItem({ - 'text/plain': Promise.resolve(str).then((text) => new Blob([text], { type: 'text/plain' })), - }) - navigator.clipboard.write([text]).then(callback) - } else { - // NOTE: Firefox has support for ClipboardItem and navigator.clipboard.write, - // but those are behind `dom.events.asyncClipboard.clipboardItem` preference. - // Good news is that other than Safari, Firefox does not care about - // Clipboard API being used async in a Promise. - Promise.resolve(str) - .then((text) => navigator.clipboard?.writeText(text)) - .then(callback) - } - } else { - toast.error('Unable to copy to clipboard') - } -} - export const detectBrowser = () => { if (!navigator) return undefined diff --git a/apps/studio/pages/new/index.tsx b/apps/studio/pages/new/index.tsx index 01395da9a8f..4c98990860f 100644 --- a/apps/studio/pages/new/index.tsx +++ b/apps/studio/pages/new/index.tsx @@ -1,7 +1,4 @@ import HCaptcha from '@hcaptcha/react-hcaptcha' -import { Elements } from '@stripe/react-stripe-js' -import { loadStripe } from '@stripe/stripe-js' -import { useTheme } from 'next-themes' import { useCallback, useEffect, useState } from 'react' import { NewOrgForm } from 'components/interfaces/Organization' @@ -9,24 +6,21 @@ import AppLayout from 'components/layouts/AppLayout/AppLayout' import DefaultLayout from 'components/layouts/DefaultLayout' import WizardLayout from 'components/layouts/WizardLayout' import { SetupIntentResponse, useSetupIntent } from 'data/stripe/setup-intent-mutation' -import { STRIPE_PUBLIC_KEY } from 'lib/constants' import { useIsHCaptchaLoaded } from 'stores/hcaptcha-loaded-store' import type { NextPageWithLayout } from 'types' -const stripePromise = loadStripe(STRIPE_PUBLIC_KEY) - /** * No org selected yet, create a new one */ const Wizard: NextPageWithLayout = () => { - const { resolvedTheme } = useTheme() - const [intent, setIntent] = useState() const captchaLoaded = useIsHCaptchaLoaded() const [captchaToken, setCaptchaToken] = useState(null) const [captchaRef, setCaptchaRef] = useState(null) + const [selectedPlan, setSelectedPlan] = useState(null) + const { mutate: setupIntent } = useSetupIntent({ onSuccess: (res) => setIntent(res) }) const captchaRefCallback = useCallback((node: any) => { @@ -37,16 +31,15 @@ const Wizard: NextPageWithLayout = () => { if (!hcaptchaToken) return console.error('Hcaptcha token is required') // Force a reload of Elements, necessary for Stripe + // Also mitigates card testing to some extent as we generate a new captcha token setIntent(undefined) setupIntent({ hcaptchaToken }) } - const options = { - clientSecret: intent ? intent.client_secret : '', - appearance: { theme: resolvedTheme?.includes('dark') ? 'night' : 'flat', labels: 'floating' }, - } as const + const loadPaymentForm = async (force = false) => { + if (selectedPlan == null || selectedPlan === 'FREE') return + if (intent != null && !force) return - const loadPaymentForm = async () => { if (captchaRef && captchaLoaded) { let token = captchaToken @@ -66,10 +59,11 @@ const Wizard: NextPageWithLayout = () => { useEffect(() => { loadPaymentForm() - }, [captchaRef, captchaLoaded]) + }, [captchaRef, captchaLoaded, selectedPlan]) const resetSetupIntent = () => { - return loadPaymentForm() + setIntent(undefined) + return loadPaymentForm(true) } const onLocalCancel = () => { @@ -96,11 +90,11 @@ const Wizard: NextPageWithLayout = () => { }} /> - {intent && ( - - resetSetupIntent()} /> - - )} + resetSetupIntent()} + onPlanSelected={(plan) => setSelectedPlan(plan)} + /> ) } diff --git a/apps/studio/pages/organizations.tsx b/apps/studio/pages/organizations.tsx index 6bd56aa5e3a..e95006b29b2 100644 --- a/apps/studio/pages/organizations.tsx +++ b/apps/studio/pages/organizations.tsx @@ -3,6 +3,7 @@ import Link from 'next/link' import { useRouter } from 'next/router' import { useEffect, useState } from 'react' +import { useParams } from 'common' import AppLayout from 'components/layouts/AppLayout/AppLayout' import DefaultLayout from 'components/layouts/DefaultLayout' import { ScaffoldContainerLegacy, ScaffoldTitle } from 'components/layouts/Scaffold' @@ -14,12 +15,21 @@ import { useProjectsQuery } from 'data/projects/projects-query' import { useIsFeatureEnabled } from 'hooks/misc/useIsFeatureEnabled' import { withAuth } from 'hooks/misc/withAuth' import { NextPageWithLayout } from 'types' -import { Button, Skeleton } from 'ui' +import { + AlertDescription_Shadcn_, + AlertTitle_Shadcn_, + Alert_Shadcn_, + Button, + CriticalIcon, + Skeleton, +} from 'ui' import { Input } from 'ui-patterns/DataInputs/Input' const OrganizationsPage: NextPageWithLayout = () => { const router = useRouter() const [search, setSearch] = useState('') + const { error: orgNotFoundError, org: orgSlug } = useParams() + const orgNotFound = orgNotFoundError === 'org_not_found' const { data: projects = [] } = useProjectsQuery() const { data: organizations = [], error, isLoading, isError, isSuccess } = useOrganizationsQuery() @@ -34,29 +44,51 @@ const OrganizationsPage: NextPageWithLayout = () => { useEffect(() => { // If there are no organizations, force the user to create one - if (isSuccess && organizations.length <= 0) { + // unless the user is on the not found page + if (isSuccess && organizations.length <= 0 && !orgNotFound) { router.push('/new') } }, [isSuccess, organizations]) return ( + {orgNotFound && ( + + + Organization not found + + That organization ({orgSlug}) does not exist or you don't have access to + it. + + + If you think this is an error, please reach out to the org owner to get access. + + + )} + Your Organizations + {organizations.length === 0 && orgNotFound && ( +

You don't have any organizations yet. Create one to get started.

+ )} +
{organizationCreationEnabled && ( )} - } - className="w-full flex-1 md:w-64 [&>div>div>div>input]:!pl-7 [&>div>div>div>div]:!pl-2" - value={search} - onChange={(event) => setSearch(event.target.value)} - /> + + {organizations.length > 0 && ( + } + className="w-full flex-1 md:w-64 [&>div>div>div>input]:!pl-7 [&>div>div>div>div]:!pl-2" + value={search} + onChange={(event) => setSearch(event.target.value)} + /> + )}
{search.length > 0 && filteredOrganizations.length === 0 && ( diff --git a/apps/studio/pages/support/new.tsx b/apps/studio/pages/support/new.tsx index 5f3a76beffb..9f94fccad75 100644 --- a/apps/studio/pages/support/new.tsx +++ b/apps/studio/pages/support/new.tsx @@ -3,6 +3,7 @@ import Link from 'next/link' import { useState } from 'react' import SVG from 'react-inlinesvg' +import { AIAssistantOption } from 'components/interfaces/Support/AIAssistantOption' import Success from 'components/interfaces/Support/Success' import { SupportFormV2 } from 'components/interfaces/Support/SupportFormV2' import AppLayout from 'components/layouts/AppLayout/AppLayout' @@ -15,8 +16,7 @@ import { withAuth } from 'hooks/misc/withAuth' import { BASE_PATH } from 'lib/constants' import { toast } from 'sonner' import { NextPageWithLayout } from 'types' -import { Button, Tooltip, TooltipContent, TooltipTrigger } from 'ui' -import { AIAssistantOption } from 'components/interfaces/Support/AIAssistantOption' +import { Button, copyToClipboard, Tooltip, TooltipContent, TooltipTrigger } from 'ui' const SupportPage: NextPageWithLayout = () => { const [sentCategory, setSentCategory] = useState() @@ -118,9 +118,7 @@ const SupportPage: NextPageWithLayout = () => { type="text" text="support@supabase.com" iconOnly - onClick={() => { - toast.success('Copied to clipboard') - }} + onClick={() => toast.success('Copied to clipboard')} />

@@ -139,7 +137,7 @@ const SupportPage: NextPageWithLayout = () => {