Skip to content

Commit

Permalink
Rename "service overview" to "service inventory" (#81933)
Browse files Browse the repository at this point in the history
To remove confusion when implementing #81718, rename the existing "service overview" to "service inventory."

* Rename `ServiceOverviewLink` to `ServiceInventoryLink`
* Remove service inventory test snapshots
* Rename `ServiceOverview` to `ServiceInventory`
* Add `service_inventory` to pageview tracking
* Rename i18n keys for service inventory
* Change "return to overview" link to "return to inventory" on settings page
* Rename `fetchOverviewPageData` to `fetchObservabilityOverviewPageData`
  • Loading branch information
smith committed Oct 30, 2020
1 parent 13858ee commit 877ad5a
Show file tree
Hide file tree
Showing 23 changed files with 106 additions and 883 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ import { loginAndWaitForPage } from '../../integration/helpers';
export const DEFAULT_TIMEOUT = 60 * 1000;

Given(`a user browses the APM UI application`, () => {
// open service overview page
// Open service inventory page
loginAndWaitForPage(`/app/apm/services`, {
from: '2020-06-01T14:59:32.686Z',
to: '2020-06-16T16:59:36.219Z',
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ import { verifyClientMetrics } from './client_metrics_helper';
export const DEFAULT_TIMEOUT = { timeout: 60 * 1000 };

Given(`a user browses the APM UI application for RUM Data`, () => {
// open service overview page
// Open UX landing page
const RANGE_FROM = 'now-24h';
const RANGE_TO = 'now';
loginAndWaitForPage(
Expand Down
10 changes: 5 additions & 5 deletions x-pack/plugins/apm/public/components/app/Home/index.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -20,12 +20,12 @@ import { ApmHeader } from '../../shared/ApmHeader';
import { EuiTabLink } from '../../shared/EuiTabLink';
import { AnomalyDetectionSetupLink } from '../../shared/Links/apm/AnomalyDetectionSetupLink';
import { ServiceMapLink } from '../../shared/Links/apm/ServiceMapLink';
import { ServiceOverviewLink } from '../../shared/Links/apm/ServiceOverviewLink';
import { ServiceInventoryLink } from '../../shared/Links/apm/service_inventory_link';
import { SettingsLink } from '../../shared/Links/apm/SettingsLink';
import { TraceOverviewLink } from '../../shared/Links/apm/TraceOverviewLink';
import { SetupInstructionsLink } from '../../shared/Links/SetupInstructionsLink';
import { ServiceMap } from '../ServiceMap';
import { ServiceOverview } from '../ServiceOverview';
import { ServiceInventory } from '../service_inventory';
import { TraceOverview } from '../TraceOverview';
import { AlertingPopoverAndFlyout } from './alerting_popover_flyout';

Expand All @@ -37,13 +37,13 @@ function getHomeTabs({
const homeTabs = [
{
link: (
<ServiceOverviewLink>
<ServiceInventoryLink>
{i18n.translate('xpack.apm.home.servicesTabLabel', {
defaultMessage: 'Services',
})}
</ServiceOverviewLink>
</ServiceInventoryLink>
),
render: () => <ServiceOverview />,
render: () => <ServiceInventory />,
name: 'services',
},
{
Expand Down

This file was deleted.

This file was deleted.

Loading

0 comments on commit 877ad5a

Please sign in to comment.