Improve home screen for limited-access users

src/plugins/home/public/application/components/manage_data/manage_data.test.tsx

@@ -88,4 +88,9 @@ describe('ManageData', () => {
  );
  expect(component).toMatchSnapshot();
  });
+
+ test('render empty without any features', () => {
+ const component = shallowWithIntl(<ManageData addBasePath={addBasePathMock} features={[]} />);
+ expect(component).toMatchSnapshot();
+ });
 });

src/plugins/home/public/application/components/manage_data/manage_data.tsx

@@ -36,31 +36,33 @@ export const ManageData: FC<Props> = ({ addBasePath, features }) => (
  <>
  {features.length > 1 && <EuiHorizontalRule margin="xl" aria-hidden="true" />}
 
- <section className="homDataManage" aria-labelledby="homDataManage__title">
- <EuiTitle size="s">
- <h2 id="homDataManage__title">
- <FormattedMessage id="home.manageData.sectionTitle" defaultMessage="Manage your data" />
- </h2>
- </EuiTitle>
+ {features.length > 1 && (
+ <section className="homDataManage" aria-labelledby="homDataManage__title">
+ <EuiTitle size="s">
+ <h2 id="homDataManage__title">
+ <FormattedMessage id="home.manageData.sectionTitle" defaultMessage="Manage your data" />
+ </h2>
+ </EuiTitle>
 
- <EuiSpacer size="m" />
+  <EuiSpacer size="m" />
 
- <EuiFlexGroup className="homDataManage__content">
- {features.map((feature) => (
- <EuiFlexItem key={feature.id}>
- <Synopsis
- id={feature.id}
- onClick={createAppNavigationHandler(feature.path)}
- description={feature.description}
- iconType={feature.icon}
- title={feature.title}
- url={addBasePath(feature.path)}
- wrapInPanel
- />
- </EuiFlexItem>
- ))}
- </EuiFlexGroup>
- </section>
+ <EuiFlexGroup className="homDataManage__content">
+ {features.map((feature) => (
+ <EuiFlexItem key={feature.id}>
+ <Synopsis
+ id={feature.id}
+ onClick={createAppNavigationHandler(feature.path)}
+ description={feature.description}
+ iconType={feature.icon}
+ title={feature.title}
+ url={addBasePath(feature.path)}
+ wrapInPanel
+ />
+ </EuiFlexItem>
+ ))}
+ </EuiFlexGroup>
+ </section>
+ )}
  </>
 );
 

src/plugins/home/public/services/feature_catalogue/feature_catalogue_registry.test.ts

@@ -88,6 +88,38 @@ describe('FeatureCatalogueRegistry', () => {
  expect(service.get()).toEqual([]);
  });
  });
+
+ describe('visibility filtering', () => {
+ test('retains items with no "visible" callback', () => {
+ const service = new FeatureCatalogueRegistry();
+ service.setup().register(DASHBOARD_FEATURE);
+ const capabilities = { catalogue: {} } as any;
+ service.start({ capabilities });
+ expect(service.get()).toEqual([DASHBOARD_FEATURE]);
+ });
+
+ test('retains items with a "visible" callback which returns "true"', () => {
+ const service = new FeatureCatalogueRegistry();
+ service.setup().register({
+ ...DASHBOARD_FEATURE,
+ visible: () => true,
+ });
+ const capabilities = { catalogue: {} } as any;
+ service.start({ capabilities });
+ expect(service.get()).toEqual([DASHBOARD_FEATURE]);
+ });
+
+ test('removes items with a "visible" callback which returns "false"', () => {
+ const service = new FeatureCatalogueRegistry();
+ service.setup().register({
+ ...DASHBOARD_FEATURE,
+ visible: () => false,
+ });
+ const capabilities = { catalogue: {} } as any;
+ service.start({ capabilities });
+ expect(service.get()).toEqual([]);
+ });
+ });
  });
 
  describe('title sorting', () => {

src/plugins/home/public/services/feature_catalogue/feature_catalogue_registry.ts

@@ -45,6 +45,8 @@ export interface FeatureCatalogueEntry {
  readonly showOnHomePage: boolean;
  /** An ordinal used to sort features relative to one another for display on the home page */
  readonly order?: number;
+ /** Optional function to control visibility of this feature. */
+ readonly visible?: () => boolean;
 }
 
 /** @public */
@@ -103,7 +105,10 @@ export class FeatureCatalogueRegistry {
  }
  const capabilities = this.capabilities;
  return [...this.features.values()]
- .filter((entry) => capabilities.catalogue[entry.id] !== false)
+ .filter(
+ (entry) =>
+ capabilities.catalogue[entry.id] !== false && (entry.visible ? entry.visible() : true)
+ )
  .sort(compareByKey('title'));
  }
 

src/plugins/management/public/plugin.ts

@@ -47,6 +47,8 @@ export class ManagementPlugin implements Plugin<ManagementSetup, ManagementStart
 
  private readonly appUpdater = new BehaviorSubject<AppUpdater>(() => ({}));
 
+ private hasAnyEnabledApps = true;
+
  constructor(private initializerContext: PluginInitializerContext) {}
 
  public setup(core: CoreSetup, { home }: ManagementSetupDependencies) {
@@ -65,6 +67,7 @@ export class ManagementPlugin implements Plugin<ManagementSetup, ManagementStart
  path: '/app/management',
  showOnHomePage: false,
  category: FeatureCatalogueCategory.ADMIN,
+ visible: () => this.hasAnyEnabledApps,
  });
  }
 
@@ -96,11 +99,11 @@ export class ManagementPlugin implements Plugin<ManagementSetup, ManagementStart
 
  public start(core: CoreStart) {
  this.managementSections.start({ capabilities: core.application.capabilities });
- const hasAnyEnabledApps = getSectionsServiceStartPrivate()
+ this.hasAnyEnabledApps = getSectionsServiceStartPrivate()
  .getSectionsEnabled()
  .some((section) => section.getAppsEnabled().length > 0);
 
- if (!hasAnyEnabledApps) {
+ if (!this.hasAnyEnabledApps) {
  this.appUpdater.next(() => {
  return {
  status: AppStatus.inaccessible,

x-pack/plugins/index_lifecycle_management/server/plugin.ts

@@ -82,10 +82,11 @@ export class IndexLifecycleManagementServerPlugin implements Plugin<void, void,
  );
 
  features.registerElasticsearchFeature({
- id: 'index_lifecycle_management',
+ id: PLUGIN.ID,
  management: {
- data: ['index_lifecycle_management'],
+ data: [PLUGIN.ID],
  },
+ catalogue: [PLUGIN.ID],
  privileges: [
  {
  requiredClusterPrivileges: ['manage_ilm'],

x-pack/plugins/snapshot_restore/server/plugin.ts

@@ -86,6 +86,7 @@ export class SnapshotRestoreServerPlugin implements Plugin<void, void, any, any>
  management: {
  data: [PLUGIN.id],
  },
+ catalogue: [PLUGIN.id],
  privileges: [
  {
  requiredClusterPrivileges: [...APP_REQUIRED_CLUSTER_PRIVILEGES],