Skip to content

Commit

Permalink
Fixes lint problems
Browse files Browse the repository at this point in the history
  • Loading branch information
michael-s-molina committed Jun 9, 2023
1 parent 4ddd593 commit 4c106d0
Show file tree
Hide file tree
Showing 7 changed files with 3 additions and 11 deletions.
3 changes: 0 additions & 3 deletions superset-frontend/src/pages/ChartList/index.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -68,7 +68,6 @@ import setupPlugins from 'src/setup/setupPlugins';
import InfoTooltip from 'src/components/InfoTooltip';
import CertifiedBadge from 'src/components/CertifiedBadge';
import { GenericLink } from 'src/components/GenericLink/GenericLink';
import getBootstrapData from 'src/utils/getBootstrapData';
import Owner from 'src/types/Owner';
import { loadTags } from 'src/components/Tags/utils';
import ChartCard from 'src/features/charts/ChartCard';
Expand Down Expand Up @@ -156,8 +155,6 @@ const StyledActions = styled.div`
color: ${({ theme }) => theme.colors.grayscale.base};
`;

const bootstrapData = getBootstrapData();

function ChartList(props: ChartListProps) {
const {
addDangerToast,
Expand Down
3 changes: 0 additions & 3 deletions superset-frontend/src/pages/DashboardList/index.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -54,7 +54,6 @@ import Dashboard from 'src/dashboard/containers/Dashboard';
import { Dashboard as CRUDDashboard } from 'src/views/CRUD/types';
import CertifiedBadge from 'src/components/CertifiedBadge';
import { loadTags } from 'src/components/Tags/utils';
import getBootstrapData from 'src/utils/getBootstrapData';
import DashboardCard from 'src/features/dashboards/DashboardCard';
import { DashboardStatus } from 'src/features/dashboards/types';

Expand Down Expand Up @@ -101,8 +100,6 @@ const Actions = styled.div`
color: ${({ theme }) => theme.colors.grayscale.base};
`;

const bootstrapData = getBootstrapData();

function DashboardList(props: DashboardListProps) {
const {
addDangerToast,
Expand Down
2 changes: 1 addition & 1 deletion superset/connectors/sqla/models.py
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,7 @@
import pandas as pd
import sqlalchemy as sa
import sqlparse
from flask import current_app, escape, Markup
from flask import escape, Markup
from flask_appbuilder import Model
from flask_babel import lazy_gettext as _
from jinja2.exceptions import TemplateError
Expand Down
1 change: 0 additions & 1 deletion superset/models/dashboard.py
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,6 @@
from typing import Any, Callable

import sqlalchemy as sqla
from flask import current_app
from flask_appbuilder import Model
from flask_appbuilder.models.decorators import renders
from flask_appbuilder.security.sqla.models import User
Expand Down
1 change: 0 additions & 1 deletion superset/models/filter_set.py
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,6 @@
import logging
from typing import Any

from flask import current_app
from flask_appbuilder import Model
from sqlalchemy import Column, ForeignKey, Integer, MetaData, String, Text
from sqlalchemy.orm import relationship
Expand Down
1 change: 0 additions & 1 deletion superset/models/slice.py
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,6 @@
from urllib import parse

import sqlalchemy as sqla
from flask import current_app
from flask_appbuilder import Model
from flask_appbuilder.models.decorators import renders
from markupsafe import escape, Markup
Expand Down
3 changes: 2 additions & 1 deletion superset/security/manager.py
Original file line number Diff line number Diff line change
Expand Up @@ -59,7 +59,6 @@
DatasetInvalidPermissionEvaluationException,
SupersetSecurityException,
)
from superset.extensions import feature_flag_manager
from superset.security.guest_token import (
GuestToken,
GuestTokenResources,
Expand Down Expand Up @@ -2005,6 +2004,8 @@ def get_rls_cache_key(self, datasource: "BaseDatasource") -> list[str]:

@staticmethod
def raise_for_user_activity_access(user_id: int) -> None:
from superset.extensions import feature_flag_manager

if not get_user_id() or (
not feature_flag_manager.is_feature_enabled("ENABLE_BROAD_ACTIVITY_ACCESS")
and user_id != get_user_id()
Expand Down

0 comments on commit 4c106d0

Please sign in to comment.