Only add this if the feature flag is enabled

chrisronline · chrisronline · commit a107da3cd7d3 · 2020-01-27T15:49:28.000-05:00
diff --git a/x-pack/legacy/plugins/monitoring/ui_exports.js b/x-pack/legacy/plugins/monitoring/ui_exports.js
@@ -6,7 +6,10 @@
 
 import { i18n } from '@kbn/i18n';
 import { resolve } from 'path';
-import { MONITORING_CONFIG_ALERTING_EMAIL_ADDRESS } from './common/constants';
+import {
+  MONITORING_CONFIG_ALERTING_EMAIL_ADDRESS,
+  KIBANA_ALERTING_ENABLED,
+} from './common/constants';
 import { DEFAULT_APP_CATEGORIES } from '../../../../src/core/utils';
 
 /**
@@ -15,32 +18,10 @@ import { DEFAULT_APP_CATEGORIES } from '../../../../src/core/utils';
  * app (injectDefaultVars and hacks)
  * @return {Object} data per Kibana plugin uiExport schema
  */
-export const getUiExports = () => ({
-  app: {
-    title: i18n.translate('xpack.monitoring.stackMonitoringTitle', {
-      defaultMessage: 'Stack Monitoring',
-    }),
-    order: 9002,
-    description: i18n.translate('xpack.monitoring.uiExportsDescription', {
-      defaultMessage: 'Monitoring for Elastic Stack',
-    }),
-    icon: 'plugins/monitoring/icons/monitoring.svg',
-    euiIconType: 'monitoringApp',
-    linkToLastSubUrl: false,
-    main: 'plugins/monitoring/monitoring',
-    category: DEFAULT_APP_CATEGORIES.management,
-  },
-  injectDefaultVars(server) {
-    const config = server.config();
-    return {
-      monitoringUiEnabled: config.get('xpack.monitoring.ui.enabled'),
-      monitoringLegacyEmailAddress: config.get(
-        'xpack.monitoring.cluster_alerts.email_notifications.email_address'
-      ),
-    };
-  },
-  uiSettingDefaults: {
-    [MONITORING_CONFIG_ALERTING_EMAIL_ADDRESS]: {
+export const getUiExports = () => {
+  const uiSettingDefaults = {};
+  if (KIBANA_ALERTING_ENABLED) {
+    uiSettingDefaults[MONITORING_CONFIG_ALERTING_EMAIL_ADDRESS] = {
       name: i18n.translate('xpack.monitoring.alertingEmailAddress.name', {
         defaultMessage: 'Alerting email address',
       }),
@@ -49,9 +30,36 @@ export const getUiExports = () => ({
         defaultMessage: `The default email address to receive alerts from Stack Monitoring`,
       }),
       category: ['monitoring'],
+    };
+  }
+
+  return {
+    app: {
+      title: i18n.translate('xpack.monitoring.stackMonitoringTitle', {
+        defaultMessage: 'Stack Monitoring',
+      }),
+      order: 9002,
+      description: i18n.translate('xpack.monitoring.uiExportsDescription', {
+        defaultMessage: 'Monitoring for Elastic Stack',
+      }),
+      icon: 'plugins/monitoring/icons/monitoring.svg',
+      euiIconType: 'monitoringApp',
+      linkToLastSubUrl: false,
+      main: 'plugins/monitoring/monitoring',
+      category: DEFAULT_APP_CATEGORIES.management,
+    },
+    injectDefaultVars(server) {
+      const config = server.config();
+      return {
+        monitoringUiEnabled: config.get('xpack.monitoring.ui.enabled'),
+        monitoringLegacyEmailAddress: config.get(
+          'xpack.monitoring.cluster_alerts.email_notifications.email_address'
+        ),
+      };
     },
-  },
-  hacks: ['plugins/monitoring/hacks/toggle_app_link_in_nav'],
-  home: ['plugins/monitoring/register_feature'],
-  styleSheetPaths: resolve(__dirname, 'public/index.scss'),
-});
+    uiSettingDefaults,
+    hacks: ['plugins/monitoring/hacks/toggle_app_link_in_nav'],
+    home: ['plugins/monitoring/register_feature'],
+    styleSheetPaths: resolve(__dirname, 'public/index.scss'),
+  };
+};
