From cee51ac084fa63eebbbfd438fb8273b5adf41e03 Mon Sep 17 00:00:00 2001
From: JzoNg <jzongcode@gmail.com>
Date: Sat, 2 Nov 2024 14:55:57 +0800
Subject: [PATCH] need author judgement

---
 .../plugin-detail-panel/action-list.tsx       |  4 +--
 web/app/components/tools/provider/detail.tsx  | 30 ++++++++++---------
 2 files changed, 18 insertions(+), 16 deletions(-)

diff --git a/web/app/components/plugins/plugin-detail-panel/action-list.tsx b/web/app/components/plugins/plugin-detail-panel/action-list.tsx
index 8c0f930251..96ca303d16 100644
--- a/web/app/components/plugins/plugin-detail-panel/action-list.tsx
+++ b/web/app/components/plugins/plugin-detail-panel/action-list.tsx
@@ -36,7 +36,7 @@ const ActionList = () => {
       <div className='mb-1 py-1'>
         <div className='mb-1 h-6 flex items-center justify-between text-text-secondary system-sm-semibold-uppercase'>
           {t('plugin.detailPanel.actionNum', { num: data.length })}
-          {providerDeclaration.is_team_authorization && (
+          {providerDeclaration.is_team_authorization && providerDeclaration.allow_delete && (
             <Button
               variant='secondary'
               size='small'
@@ -48,7 +48,7 @@ const ActionList = () => {
             </Button>
           )}
         </div>
-        {!providerDeclaration.is_team_authorization && (
+        {!providerDeclaration.is_team_authorization && providerDeclaration.allow_delete && (
           <Button
             variant='primary'
             className='w-full'
diff --git a/web/app/components/tools/provider/detail.tsx b/web/app/components/tools/provider/detail.tsx
index 8c9f70966b..96f36a2095 100644
--- a/web/app/components/tools/provider/detail.tsx
+++ b/web/app/components/tools/provider/detail.tsx
@@ -45,7 +45,7 @@ import { ConfigurationMethodEnum } from '@/app/components/header/account-setting
 import Loading from '@/app/components/base/loading'
 import { useAppContext } from '@/context/app-context'
 
-interface Props {
+type Props = {
   collection: Collection
   onHide: () => void
   onRefreshData: () => void
@@ -293,21 +293,23 @@ const ProviderDetail = ({
         <div className='pt-3'>
           {isDetailLoading && <div className='flex h-[200px]'><Loading type='app' /></div>}
           {/* Builtin type */}
-          {!isDetailLoading && (collection.type === CollectionType.builtIn) && needAuth && isAuthed && (
+          {!isDetailLoading && (collection.type === CollectionType.builtIn) && isAuthed && (
             <div className='mb-1 h-6 flex items-center justify-between text-text-secondary system-sm-semibold-uppercase'>
               {t('plugin.detailPanel.actionNum', { num: 3 })}
-              <Button
-                variant='secondary'
-                size='small'
-                onClick={() => {
-                  if (collection.type === CollectionType.builtIn || collection.type === CollectionType.model)
-                    showSettingAuthModal()
-                }}
-                disabled={!isCurrentWorkspaceManager}
-              >
-                <Indicator className='mr-2' color={'green'} />
-                {t('tools.auth.authorized')}
-              </Button>
+              {needAuth && (
+                <Button
+                  variant='secondary'
+                  size='small'
+                  onClick={() => {
+                    if (collection.type === CollectionType.builtIn || collection.type === CollectionType.model)
+                      showSettingAuthModal()
+                  }}
+                  disabled={!isCurrentWorkspaceManager}
+                >
+                  <Indicator className='mr-2' color={'green'} />
+                  {t('tools.auth.authorized')}
+                </Button>
+              )}
             </div>
           )}
           {!isDetailLoading && (collection.type === CollectionType.builtIn) && needAuth && !isAuthed && (