feat: remove newApplicationsList feature flag (#4653)

frontend/src/component/application/ApplicationList/OldApplicationList.tsx

@@ -1,99 +0,0 @@
-import { useEffect, useMemo, useState } from 'react';
-import { CircularProgress, Link } from '@mui/material';
-import { Warning } from '@mui/icons-material';
-import { AppsLinkList, styles as themeStyles } from 'component/common';
-import { PageContent } from 'component/common/PageContent/PageContent';
-import { PageHeader } from 'component/common/PageHeader/PageHeader';
-import useApplications from 'hooks/api/getters/useApplications/useApplications';
-import { ConditionallyRender } from 'component/common/ConditionallyRender/ConditionallyRender';
-import { useSearchParams } from 'react-router-dom';
-import { Search } from 'component/common/Search/Search';
-import { safeRegExp } from '@server/util/escape-regex';
-
-type PageQueryType = Partial<Record<'search', string>>;
-
-export const OldApplicationList = () => {
-    const { applications, loading } = useApplications();
-    const [searchParams, setSearchParams] = useSearchParams();
-    const [searchValue, setSearchValue] = useState(
-        searchParams.get('search') || ''
-    );
-
-    useEffect(() => {
-        const tableState: PageQueryType = {};
-        if (searchValue) {
-            tableState.search = searchValue;
-        }
-
-        setSearchParams(tableState, {
-            replace: true,
-        });
-    }, [searchValue, setSearchParams]);
-
-    const filteredApplications = useMemo(() => {
-        const regExp = safeRegExp(searchValue, 'i');
-        return searchValue
-            ? applications?.filter(a => regExp.test(a.appName))
-            : applications;
-    }, [applications, searchValue]);
-
-    const renderNoApplications = () => (
-        <>
-            <section style={{ textAlign: 'center' }}>
-                <Warning titleAccess="Warning" /> <br />
-                <br />
-                Oh snap, it does not seem like you have connected any
-                applications. To connect your application to Unleash you will
-                require a Client SDK.
-                <br />
-                <br />
-                You can read more about how to use Unleash in your application
-                in the{' '}
-                <Link href="https://docs.getunleash.io/docs/sdks/">
-                    documentation.
-                </Link>
-            </section>
-        </>
-    );
-
-    if (!filteredApplications) {
-        return <CircularProgress variant="indeterminate" />;
-    }
-
-    let applicationCount =
-        filteredApplications.length < applications.length
-            ? `${filteredApplications.length} of ${applications.length}`
-            : applications.length;
-
-    return (
-        <>
-            <PageContent
-                header={
-                    <PageHeader
-                        title={`Applications (${applicationCount})`}
-                        actions={
-                            <Search
-                                initialValue={searchValue}
-                                onChange={setSearchValue}
-                            />
-                        }
-                    />
-                }
-            >
-                <div className={themeStyles.fullwidth}>
-                    <ConditionallyRender
-                        condition={filteredApplications.length > 0}
-                        show={<AppsLinkList apps={filteredApplications} />}
-                        elseShow={
-                            <ConditionallyRender
-                                condition={loading}
-                                show={<div>...loading</div>}
-                                elseShow={renderNoApplications()}
-                            />
-                        }
-                    />
-                </div>
-            </PageContent>
-        </>
-    );
-};

frontend/src/component/application/ApplicationList/TemporaryApplicationListWrapper.tsx

@@ -1,16 +0,0 @@
-import { ConditionallyRender } from 'component/common/ConditionallyRender/ConditionallyRender';
-import useUiConfig from '../../../hooks/api/getters/useUiConfig/useUiConfig';
-import { ApplicationList } from './ApplicationList';
-import { OldApplicationList } from './OldApplicationList';
-
-export const TemporaryApplicationListWrapper = () => {
-    const { uiConfig } = useUiConfig();
-
-    return (
-        <ConditionallyRender
-            condition={Boolean(uiConfig.flags.newApplicationList)}
-            show={<ApplicationList />}
-            elseShow={<OldApplicationList />}
-        />
-    );
-};

frontend/src/component/menu/routes.ts

@@ -44,8 +44,8 @@ import { LazyProject } from 'component/project/Project/LazyProject';
 import { LoginHistory } from 'component/loginHistory/LoginHistory';
 import { FeatureTypesList } from 'component/featureTypes/FeatureTypesList';
 import { AddonsList } from 'component/integrations/IntegrationList/AddonsList';
-import { TemporaryApplicationListWrapper } from 'component/application/ApplicationList/TemporaryApplicationListWrapper';
 import { ViewIntegration } from 'component/integrations/ViewIntegration/ViewIntegration';
+import { ApplicationList } from '../application/ApplicationList/ApplicationList';
 
 export const routes: IRoute[] = [
     // Splash
@@ -181,7 +181,7 @@ export const routes: IRoute[] = [
     {
         path: '/applications',
         title: 'Applications',
-        component: TemporaryApplicationListWrapper,
+        component: ApplicationList,
         type: 'protected',
         menu: { mobile: true, advanced: true },
     },
@@ -502,14 +502,13 @@ const computeRoutes = () => {
 
 export const getCondensedRoutes = (routes: IRoute[]): INavigationMenuItem[] => {
     return routes.map(route => {
-        const condensedRoute = {
+        return {
             path: route.path,
             flag: route.flag,
             title: route.title,
             menu: route.menu,
             configFlag: route.configFlag,
         };
-        return condensedRoute;
     });
 };
 

frontend/src/interfaces/uiConfig.ts

@@ -60,7 +60,6 @@ export type UiFlags = {
     customRootRolesKillSwitch?: boolean;
     strategyVariant?: boolean;
     lastSeenByEnvironment?: boolean;
-    newApplicationList?: boolean;
     integrationsRework?: boolean;
     multipleRoles?: boolean;
     featureNamingPattern?: boolean;

src/lib/__snapshots__/create-config.test.ts.snap

@@ -94,7 +94,6 @@ exports[`should create default config 1`] = `
       },
       "migrationLock": true,
       "multipleRoles": false,
-      "newApplicationList": false,
       "personalAccessTokensKillSwitch": false,
       "proPlanAutoCharge": false,
       "responseTimeWithAppNameKillSwitch": false,
@@ -130,7 +129,6 @@ exports[`should create default config 1`] = `
       },
       "migrationLock": true,
       "multipleRoles": false,
-      "newApplicationList": false,
       "personalAccessTokensKillSwitch": false,
       "proPlanAutoCharge": false,
       "responseTimeWithAppNameKillSwitch": false,

src/lib/db/client-applications-store.ts

@@ -7,7 +7,6 @@ import {
 import { Logger, LogProvider } from '../logger';
 import { IApplicationQuery } from '../types/query';
 import { Db } from './db';
-import { IFlagResolver } from '../types';
 
 const COLUMNS = [
     'app_name',
@@ -110,18 +109,10 @@ export default class ClientApplicationsStore
 {
     private db: Db;
 
-    private flagResolver: IFlagResolver;
-
     private logger: Logger;
 
-    constructor(
-        db: Db,
-        eventBus: EventEmitter,
-        getLogger: LogProvider,
-        flagResolver: IFlagResolver,
-    ) {
+    constructor(db: Db, eventBus: EventEmitter, getLogger: LogProvider) {
         this.db = db;
-        this.flagResolver = flagResolver;
         this.logger = getLogger('client-applications-store.ts');
     }
 
@@ -193,7 +184,6 @@ export default class ClientApplicationsStore
     async getAppsForStrategy(
         query: IApplicationQuery,
     ): Promise<IClientApplication[]> {
-        if (this.flagResolver.isEnabled('newApplicationList')) {
         const rows = await this.db
             .select([
                 ...COLUMNS.map((column) => `${TABLE}.${column}`),
@@ -214,17 +204,6 @@ export default class ClientApplicationsStore
             );
         }
         return apps;
-        } else {
-            const rows = await this.db.select(COLUMNS).from(TABLE);
-            const apps = rows.map(mapRow);
-
-            if (query.strategyName) {
-                return apps.filter((app) =>
-                    app.strategies.includes(query.strategyName),
-                );
-            }
-            return apps;
-        }
     }
 
     async getUnannounced(): Promise<IClientApplication[]> {

src/lib/db/index.ts

@@ -1,5 +1,4 @@
-import { IUnleashConfig } from '../types/option';
-import { IUnleashStores } from '../types/stores';
+import { IUnleashConfig, IUnleashStores } from '../types';
 
 import EventStore from './event-store';
 import FeatureToggleStore from './feature-toggle-store';
@@ -54,7 +53,6 @@ export const createStores = (
             db,
             eventBus,
             getLogger,
-            config.flagResolver,
         ),
         clientInstanceStore: new ClientInstanceStore(db, eventBus, getLogger),
         clientMetricsStoreV2: new ClientMetricsStoreV2(

src/lib/types/experimental.ts

@@ -24,7 +24,6 @@ export type IFlagKey =
     | 'filterInvalidClientMetrics'
     | 'lastSeenByEnvironment'
     | 'customRootRolesKillSwitch'
-    | 'newApplicationList'
     | 'integrationsRework'
     | 'multipleRoles'
     | 'featureNamingPattern'
@@ -120,10 +119,6 @@ const flags: IFlags = {
         process.env.UNLEASH_EXPERIMENTAL_MULTIPLE_ROLES,
         false,
     ),
-    newApplicationList: parseEnvVarBoolean(
-        process.env.UNLEASH_EXPERIMENTAL_NEW_APPLICATION_LIST,
-        false,
-    ),
     featureNamingPattern: parseEnvVarBoolean(
         process.env.UNLEASH_EXPERIMENTAL_FEATURE_NAMING_PATTERN,
         false,

src/server-dev.ts

@@ -40,7 +40,6 @@ process.nextTick(async () => {
                         slackAppAddon: true,
                         lastSeenByEnvironment: true,
                         integrationsRework: true,
-                        newApplicationList: true,
                         featureNamingPattern: true,
                         doraMetrics: true,
                     },

src/test/e2e/api/admin/metrics.e2e.test.ts

@@ -9,16 +9,8 @@ let app: IUnleashTest;
 let db: ITestDb;
 
 beforeAll(async () => {
-    db = await dbInit('metrics_serial', getLogger, {
-        experimental: { flags: { newApplicationList: true } },
-    });
-    app = await setupAppWithCustomConfig(db.stores, {
-        experimental: {
-            flags: {
-                newApplicationList: true,
-            },
-        },
-    });
+    db = await dbInit('metrics_serial', getLogger, {});
+    app = await setupAppWithCustomConfig(db.stores, {});
 });
 
 beforeEach(async () => {