Skip to content

Commit

Permalink
[Behavioral Analytics] Change Analytics DNS path (#147934)
Browse files Browse the repository at this point in the history
### Description
Recently it was discovered that our initial path to analytics brings
some inconsistency into Enterprise Search routes, so it was decided to
replace the `analytics/api` path with `api/analytics`.

This PR is dedicated to changing Analytics DNS path from `analytics/api`
to `api/analytics`
  • Loading branch information
tutelaris authored Dec 21, 2022
1 parent af2a7d6 commit 3697838
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,7 @@ describe('AnalyticsCollectionIntegrate', () => {
<AnalyticsCollectionIntegrate collection={analyticsCollections} />
);
expect(wrapper.find(EuiCodeBlock).at(0).text()).toContain(
'data-dsn="/analytics/api/collections/1"'
'data-dsn="/api/analytics/collections/1"'
);
expect(wrapper.find(EuiCodeBlock).at(0).text()).toContain('src="/analytics.js"');
});
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ export type TabKey = 'javascriptEmbed' | 'searchuiEmbed' | 'javascriptClientEmbe
export const AnalyticsCollectionIntegrate: React.FC<AnalyticsCollectionIntegrateProps> = ({
collection,
}) => {
const analyticsDNSUrl = getEnterpriseSearchUrl(`/analytics/api/collections/${collection.id}`);
const analyticsDNSUrl = getEnterpriseSearchUrl(`/api/analytics/collections/${collection.id}`);
const webClientSrc = getEnterpriseSearchUrl('/analytics.js');

const [selectedTab, setSelectedTab] = React.useState<TabKey>('javascriptEmbed');
Expand Down

0 comments on commit 3697838

Please sign in to comment.