dima/supabase
1
0 Fork 0
mirror of https://github.com/supabase/supabase.git synced 2026-10-18 23:59:08 +03:00
Code Issues Packages Projects Releases Wiki Activity

style: drop the verified badge from the consent screen

Browse Source 
The tick after the app name is gone while `is_verified` stays on the request
contract and in the mocks. An unverified publisher still raises the trust
warning.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
This commit is contained in:
kemalandClaude Opus 5 authored and Gildas Garcia committed 2026-10-01 11:29:08 +02:00
1 parent 2ed549c94b
commit a573d3568d
2 files changed
+5 -13
Download Patch File Download Diff File Expand all files Collapse all files

No files matched your search

apps/studio/components/interfaces/Organization/OAuthApps/Consent/OAuthAppsAuthorizeScreen.test.tsx
+4 -4
View File
@@ -41,7 +41,7 @@ describe('OAuthAppsAuthorizeScreen', () => {
).toBeInTheDocument()
})
test('renders the publisher warning and no verified tick for the unverified fixture', async () => {
test('renders the publisher warning for the unverified fixture', async () => {
customRender(<OAuthAppsAuthorizeScreen mockState="unverified" navigate={vi.fn()} />)
expect(
@@ -49,18 +49,18 @@ describe('OAuthAppsAuthorizeScreen', () => {
"This publisher isn't verified by Supabase. Only continue if you trust it."
)
).toBeInTheDocument()
expect(screen.queryByRole('img', { name: 'Verified' })).not.toBeInTheDocument()
})
test('shows the verified tick and no publisher warning for a verified fixture', async () => {
test('shows no publisher warning and no verified tick for a verified fixture', async () => {
customRender(<OAuthAppsAuthorizeScreen mockState="ideal" navigate={vi.fn()} />)
expect(await screen.findByRole('img', { name: 'Verified' })).toBeInTheDocument()
await screen.findByText('Permissions requested')
expect(
screen.queryByText(
"This publisher isn't verified by Supabase. Only continue if you trust it."
)
).not.toBeInTheDocument()
expect(screen.queryByRole('img', { name: 'Verified' })).not.toBeInTheDocument()
})
test('renders the success screen after the approve mutation resolves', async () => {
apps/studio/components/interfaces/Organization/OAuthApps/Consent/OAuthAppsAuthorizeScreen.tsx
+1 -9
View File
@@ -1,4 +1,3 @@
import { BadgeCheck } from 'lucide-react'
import { useState } from 'react'
import { Button } from 'ui'
import { Admonition } from 'ui-patterns/Admonition'
@@ -157,14 +156,7 @@ export const OAuthAppsAuthorizeScreen = ({
logo={
<LogoPair left={<DestinationLogo name={request.app_name} />} right={<SupabaseLogo />} />
}
title={
<span className="inline-flex items-center gap-1.5">
Authorize {request.app_name}
{request.is_verified && (
<BadgeCheck role="img" aria-label="Verified" className="size-4 text-brand" />
)}
</span>
}
title={`Authorize ${request.app_name}`}
description="This application wants to access your Supabase Account"
>
<div className="flex flex-col gap-6 px-6 pb-6">