Skip to content

Commit

Permalink
Add clicks for setup mode
Browse files Browse the repository at this point in the history
  • Loading branch information
chrisronline committed Sep 16, 2020
1 parent 8cf79cb commit 811cbe6
Show file tree
Hide file tree
Showing 7 changed files with 24 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,7 @@ import React from 'react';
import { EuiButton } from '@elastic/eui';
import { i18n } from '@kbn/i18n';
import './enter_button.scss';
import { METRIC_TYPE, useUiTracker } from '../../../../observability/public';

export interface SetupModeEnterButtonProps {
enabled: boolean;
Expand All @@ -18,6 +19,7 @@ export const SetupModeEnterButton: React.FC<SetupModeEnterButtonProps> = (
props: SetupModeEnterButtonProps
) => {
const [isLoading, setIsLoading] = React.useState(false);
const trackStat = useUiTracker({ app: 'stack_monitoring' });

if (!props.enabled) {
return null;
Expand All @@ -26,6 +28,7 @@ export const SetupModeEnterButton: React.FC<SetupModeEnterButtonProps> = (
async function enterSetupMode() {
setIsLoading(true);
await props.toggleSetupMode(true);
trackStat({ metric: `btnclick_setupmode`, metricType: METRIC_TYPE.CLICK });
setIsLoading(false);
}

Expand Down
12 changes: 11 additions & 1 deletion x-pack/plugins/monitoring/public/lib/setup_mode.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,7 @@ import React from 'react';
import { render } from 'react-dom';
import { get, includes } from 'lodash';
import { i18n } from '@kbn/i18n';
import { KibanaContextProvider } from '../../../../../src/plugins/kibana_react/public';
import { Legacy } from '../legacy_shims';
import { ajaxErrorHandlersProvider } from './ajax_error_handler';
import { SetupModeEnterButton } from '../components/setup_mode/enter_button';
Expand Down Expand Up @@ -179,8 +180,17 @@ export const setSetupModeMenuItem = () => {
const globalState = angularState.injector.get('globalState');
const enabled = !globalState.inSetupMode;

const services = {
usageCollection: Legacy.shims.usageCollection,
};
const I18nContext = Legacy.shims.I18nContext;

render(
<SetupModeEnterButton enabled={enabled} toggleSetupMode={toggleSetupMode} />,
<KibanaContextProvider services={services}>
<I18nContext>
<SetupModeEnterButton enabled={enabled} toggleSetupMode={toggleSetupMode} />
</I18nContext>
</KibanaContextProvider>,
document.getElementById('setupModeNav')
);
};
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -64,6 +64,7 @@ uiRoutes.when('/elasticsearch/indices/:index/advanced', {
indexName,
},
}),
telemetryPageViewTitle: 'elasticsearch_index_advanced',
defaultData: {},
getPageData,
reactNodeId: 'monitoringElasticsearchAdvancedIndexApp',
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -66,6 +66,7 @@ uiRoutes.when('/elasticsearch/indices/:index', {
indexName,
},
}),
telemetryPageViewTitle: 'elasticsearch_index',
defaultData: {},
getPageData,
reactNodeId: 'monitoringElasticsearchIndexApp',
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -60,6 +60,7 @@ uiRoutes.when('/elasticsearch/nodes/:node/advanced', {
defaultData: {},
getPageData,
reactNodeId: 'monitoringElasticsearchAdvancedNodeApp',
telemetryPageViewTitle: 'elasticsearch_node_advanced',
$scope,
$injector,
alerts: {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -42,6 +42,7 @@ uiRoutes.when('/elasticsearch/nodes/:node', {
nodeName,
},
}),
telemetryPageViewTitle: 'elasticsearch_node',
defaultData: {},
getPageData,
reactNodeId: 'monitoringElasticsearchNodeApp',
Expand Down
7 changes: 6 additions & 1 deletion x-pack/plugins/observability/typings/common.ts
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,12 @@
* you may not use this file except in compliance with the Elastic License.
*/

export type ObservabilityApp = 'infra_metrics' | 'infra_logs' | 'apm' | 'uptime';
export type ObservabilityApp =
| 'infra_metrics'
| 'infra_logs'
| 'apm'
| 'uptime'
| 'stack_monitoring';

export type PromiseReturnType<Func> = Func extends (...args: any[]) => Promise<infer Value>
? Value
Expand Down

0 comments on commit 811cbe6

Please sign in to comment.