Skip to content

Commit c43f99c

Browse files
committed
fix(marketplace): remove dynamic-plugins-info plugins
Signed-off-by: Yi Cai <[email protected]>
1 parent c1969a0 commit c43f99c

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

49 files changed

+23
-1647
lines changed

dynamic-plugins.default.yaml

Lines changed: 14 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -511,6 +511,9 @@ plugins:
511511
menuItems:
512512
default.create:
513513
title: ''
514+
default.admin:
515+
title: Administration
516+
icon: admin
514517
red-hat-developer-hub.backstage-plugin-global-header:
515518
mountPoints:
516519
- mountPoint: application/header
@@ -1197,17 +1200,20 @@ plugins:
11971200
frontend:
11981201
red-hat-developer-hub.backstage-plugin-marketplace:
11991202
appIcons:
1200-
- name: marketplace
1201-
importName: MarketplaceIcon
1203+
- name: pluginsIcon
1204+
importName: PluginsIcon
12021205
dynamicRoutes:
1203-
- path: /extensions/catalog
1206+
- path: /marketplace
12041207
importName: DynamicMarketplacePluginRouter
1208+
menuItem:
1209+
icon: pluginsIcon
1210+
text: Extensions
12051211
mountPoints:
1206-
- mountPoint: internal.plugins/tab
1207-
importName: DynamicMarketplacePluginContent
1208-
config:
1209-
path: marketplace
1210-
title: Catalog
1212+
- mountPoint: application/provider
1213+
importName: InstallationContextProvider
1214+
menuItems:
1215+
marketplace:
1216+
parent: default.admin
12111217

12121218
# Group: Orchestrator
12131219
- package: "@redhat/[email protected]"

packages/app/package.json

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,6 @@
4141
"@backstage/theme": "0.6.6",
4242
"@emotion/react": "11.14.0",
4343
"@emotion/styled": "11.14.1",
44-
"@internal/plugin-dynamic-plugins-info": "*",
4544
"@mui/icons-material": "5.18.0",
4645
"@mui/material": "5.18.0",
4746
"@mui/styled-engine": "5.18.0",

packages/app/src/App.tsx

Lines changed: 0 additions & 49 deletions
Original file line numberDiff line numberDiff line change
@@ -1,14 +1,9 @@
11
import GlobalStyles from '@mui/material/GlobalStyles';
22

33
import { apis } from './apis';
4-
import { StaticPlugins } from './components/DynamicRoot/DynamicRoot';
54
import ScalprumRoot from './components/DynamicRoot/ScalprumRoot';
65
import { DefaultMainMenuItems } from './consts';
76

8-
// Statically integrated frontend plugins
9-
const { dynamicPluginsInfoPlugin, ...dynamicPluginsInfoPluginModule } =
10-
await import('@internal/plugin-dynamic-plugins-info');
11-
127
// The base UI configuration, these values can be overridden by values
138
// specified in external configuration files
149
const baseFrontendConfig = {
@@ -17,62 +12,18 @@ const baseFrontendConfig = {
1712
dynamicPlugins: {
1813
frontend: {
1914
'default.main-menu-items': DefaultMainMenuItems,
20-
// please keep this in sync with plugins/dynamic-plugins-info/app-config.janus-idp.yaml
21-
'internal.plugin-dynamic-plugins-info': {
22-
appIcons: [
23-
{ name: 'pluginsInfoIcon', importName: 'PluginsInfoIcon' },
24-
{ name: 'adminIcon', importName: 'AdminIcon' },
25-
],
26-
dynamicRoutes: [
27-
{
28-
path: '/extensions',
29-
importName: 'DynamicPluginsInfoPage',
30-
menuItem: { text: 'Plugins', icon: 'pluginsInfoIcon' },
31-
},
32-
],
33-
mountPoints: [
34-
{
35-
mountPoint: 'internal.plugins/tab',
36-
importName: 'DynamicPluginsInfoContent',
37-
config: {
38-
path: 'installed',
39-
title: 'Installed',
40-
},
41-
},
42-
],
43-
menuItems: {
44-
admin: {
45-
title: 'Administration',
46-
icon: 'adminIcon',
47-
},
48-
extensions: {
49-
parent: 'admin',
50-
title: 'Extensions',
51-
icon: 'pluginsInfoIcon',
52-
},
53-
},
54-
},
5515
},
5616
},
5717
},
5818
};
5919

60-
// The map of static plugins by package name
61-
const staticPlugins: StaticPlugins = {
62-
'internal.plugin-dynamic-plugins-info': {
63-
plugin: dynamicPluginsInfoPlugin,
64-
module: dynamicPluginsInfoPluginModule,
65-
},
66-
};
67-
6820
const AppRoot = () => (
6921
<>
7022
<GlobalStyles styles={{ html: { overflowY: 'hidden' } }} />
7123
<ScalprumRoot
7224
apis={apis}
7325
afterInit={() => import('./components/AppBase')}
7426
baseFrontendConfig={baseFrontendConfig}
75-
plugins={staticPlugins}
7627
/>
7728
</>
7829
);

packages/app/src/components/DynamicRoot/CommonIcons.tsx

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,7 @@ import Extension from '@mui/icons-material/ExtensionOutlined';
88
import Favorite from '@mui/icons-material/Favorite';
99
import FolderOpen from '@mui/icons-material/FolderOpenOutlined';
1010
import GppGood from '@mui/icons-material/GppGoodOutlined';
11+
import AdminIcon from '@mui/icons-material/GppMaybeOutlined';
1112
import Help from '@mui/icons-material/HelpOutline';
1213
import Home from '@mui/icons-material/HomeOutlined';
1314
import InsertChart from '@mui/icons-material/InsertChartOutlined';
@@ -61,6 +62,7 @@ const CommonIcons: {
6162
logout: Logout,
6263
developerHub: DeveloperHub,
6364
account: AccountCircleOutlinedIcon,
65+
admin: AdminIcon,
6466
};
6567

6668
export default CommonIcons;

packages/app/src/components/Root/Root.tsx

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,6 @@ import { SidebarSearchModal } from '@backstage/plugin-search';
2424
import { Settings as SidebarSettings } from '@backstage/plugin-user-settings';
2525

2626
import { policyEntityCreatePermission } from '@backstage-community/plugin-rbac-common';
27-
import { AdminIcon } from '@internal/plugin-dynamic-plugins-info';
2827
import AccountCircleOutlinedIcon from '@mui/icons-material/AccountCircleOutlined';
2928
import ChevronRightIcon from '@mui/icons-material/ChevronRight';
3029
import ExpandMore from '@mui/icons-material/ExpandMore';
@@ -409,8 +408,8 @@ export const Root = ({ children }: PropsWithChildren<{}>) => {
409408
: menuItems.filter(mi => !mi.name.startsWith('default.'));
410409

411410
menuItemArray = isBottomMenuSection
412-
? menuItemArray.filter(mi => mi.name === 'admin')
413-
: menuItemArray.filter(mi => mi.name !== 'admin');
411+
? menuItemArray.filter(mi => mi.name.includes('admin'))
412+
: menuItemArray.filter(mi => !mi.name.includes('admin'));
414413

415414
if (isBottomMenuSection && !canDisplayRBACMenuItem && !loadingPermission) {
416415
menuItemArray[0].children = menuItemArray[0].children?.filter(
@@ -497,8 +496,8 @@ export const Root = ({ children }: PropsWithChildren<{}>) => {
497496
{showAdministration && (
498497
<>
499498
<SidebarDivider />
500-
<SidebarGroup label="Administration" icon={<AdminIcon />}>
501-
{renderMenuItems(false, true)}
499+
<SidebarGroup label="Administration">
500+
{renderMenuItems(true, true)}
502501
</SidebarGroup>
503502
</>
504503
)}

packages/app/src/utils/dynamicUI/initializeRemotePlugins.ts

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -4,10 +4,7 @@ import { ScalprumState } from '@scalprum/react-core';
44
import { RemotePlugins } from '../../components/DynamicRoot/DynamicRoot';
55

66
// See packages/app/src/App.tsx
7-
const ignoreStaticPlugins = [
8-
'default.main-menu-items',
9-
'internal.plugin-dynamic-plugins-info',
10-
];
7+
const ignoreStaticPlugins = ['default.main-menu-items'];
118

129
const initializeRemotePlugins = async (
1310
pluginStore: ScalprumState['pluginStore'],

packages/backend/package.json

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -65,7 +65,6 @@
6565
"@backstage/plugin-search-backend-module-catalog": "0.3.4",
6666
"@backstage/plugin-search-backend-module-pg": "0.5.44",
6767
"@backstage/plugin-user-settings-backend": "^0.3.5",
68-
"@internal/plugin-dynamic-plugins-info-backend": "*",
6968
"@internal/plugin-licensed-users-info-backend": "*",
7069
"@internal/plugin-scalprum-backend": "*",
7170
"@janus-idp/backstage-plugin-audit-log-node": "1.7.2",

packages/backend/src/index.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -159,7 +159,6 @@ if (process.env.ENABLE_AUTH_PROVIDER_MODULE_OVERRIDE !== 'true') {
159159
staticLogger.info(`Default authentication provider module disabled`);
160160
}
161161

162-
backend.add(import('@internal/plugin-dynamic-plugins-info-backend'));
163162
backend.add(import('@internal/plugin-scalprum-backend'));
164163
backend.add(import('@internal/plugin-licensed-users-info-backend'));
165164

plugins/dynamic-plugins-info-backend/.eslintrc.js

Lines changed: 0 additions & 1 deletion
This file was deleted.

plugins/dynamic-plugins-info-backend/.lintstagedrc.json

Lines changed: 0 additions & 4 deletions
This file was deleted.

0 commit comments

Comments
 (0)