QT/superset2
1
0
Fork 0
mirror of https://github.com/apache/superset.git synced 2026-06-01 13:49:21 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix: Hide View in SQL Lab for users without access (#34293)

Browse Source
This commit is contained in:
Vitor Avila
2025-07-24 10:45:31 -03:00
committed by GitHub
parent da55a6c94a
commit c99843b13a
2 changed files with 45 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
superset-frontend/src/explore/components/controls/ViewQuery.tsx
View File

@@ -24,11 +24,14 @@ import {
useEffect,
useState,
} from 'react';
import { useSelector } from 'react-redux';
import rison from 'rison';
import { styled, SupersetClient, t } from '@superset-ui/core';
import { Icons, Switch, Button, Skeleton } from '@superset-ui/core/components';
import { CopyToClipboard } from 'src/components';
import { RootState } from 'src/dashboard/types';
import { CopyButton } from 'src/explore/components/DataTableControl';
import { findPermission } from 'src/utils/findPermission';
import CodeSyntaxHighlighter, {
SupportedLanguage,
preloadLanguages,
@@ -89,6 +92,9 @@ const ViewQuery: FC<ViewQueryProps> = props => {
const [showFormatSQL, setShowFormatSQL] = useState(true);
const history = useHistory();
const currentSQL = (showFormatSQL ? formattedSQL : sql) ?? sql;
const canAccessSQLLab = useSelector((state: RootState) =>
findPermission('menu_access', 'SQL Lab', state.user?.roles),
);
// Preload the language when component mounts to ensure smooth experience
useEffect(() => {
@@ -162,7 +168,9 @@ const ViewQuery: FC<ViewQueryProps> = props => {
</CopyButtonViewQuery>
}
/>
<Button onClick={navToSQLLab}>{t('View in SQL Lab')}</Button>
{canAccessSQLLab && (
<Button onClick={navToSQLLab}>{t('View in SQL Lab')}</Button>
)}
</StyledHeaderActionContainer>
<StyledHeaderActionContainer>
<Switch