From ca87bc495d55abedd840d543ea3f1433fcafba9e Mon Sep 17 00:00:00 2001 From: John Schulz Date: Thu, 14 Jan 2021 08:53:16 -0500 Subject: [PATCH] Fix broken icon key for Maps (#88093) ## PR Summary Show [`emsApp` icon from EUI](https://eui.elastic.co/#/display/icons#apps) if integration includes map assets. Screen Shot 2021-01-12 at 3 14 14 PM Looking at https://github.com/elastic/kibana/pull/62059#discussion_r401211966, I believe it landed with the incorrect key at a time when that part of the page wasn't visible. When we enabled that section, we likely didn't notice the missing key since most integrations don't have Maps assets ## Bug: `master` doesn't have an icon for Map assets 1. Visit an integration with map assets, like AWS http://localhost:5601/app/fleet#/integrations/detail/aws-0.3.16 2. Observe missing icon for Map in right column summary & HTTP request for `/app/mapApp` Screen Shot 2021-01-12 at 3 10 44 PM ## This PR adds one 1. Visit an integration with map assets, like AWS http://localhost:5601/app/fleet#/integrations/detail/aws-0.3.16 2. Observe EMS icon is present in right column summary Screen Shot 2021-01-12 at 3 11 39 PM --- .../fleet/public/applications/fleet/sections/epm/constants.tsx | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/x-pack/plugins/fleet/public/applications/fleet/sections/epm/constants.tsx b/x-pack/plugins/fleet/public/applications/fleet/sections/epm/constants.tsx index 533026aa93f7..01f9ecb2723a 100644 --- a/x-pack/plugins/fleet/public/applications/fleet/sections/epm/constants.tsx +++ b/x-pack/plugins/fleet/public/applications/fleet/sections/epm/constants.tsx @@ -40,7 +40,7 @@ export const AssetIcons: Record = { index_pattern: 'indexPatternApp', search: 'searchProfilerApp', visualization: 'visualizeApp', - map: 'mapApp', + map: 'emsApp', }; export const ServiceIcons: Record = {