Skip to content

Commit e5270a9

Browse files
authored
fix: change appicon name matching logic to lowercase both sides of th… (#1268)
* fix: change appicon name matching logic to lowercase both sides of the match * fix: make lowerCaseName const
1 parent a6547ae commit e5270a9

1 file changed

Lines changed: 12 additions & 3 deletions

File tree

src/molecules/ApplicationIcon/ApplicationIcon.tsx

Lines changed: 12 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -50,7 +50,10 @@ interface ApplicationIconData {
5050
}
5151
const apps: ApplicationIconData[] = [
5252
{ appName: ['adca'], component: Adca },
53-
{ appName: ['portal', 'embark', 'jsembark'], component: JsEmbark },
53+
{
54+
appName: ['portal', 'embark', 'jsembark', 'js embark'],
55+
component: JsEmbark,
56+
},
5457
{ appName: ['acquire'], component: Acquire },
5558
{ appName: ['dasha'], component: Dasha },
5659
{
@@ -62,7 +65,10 @@ const apps: ApplicationIconData[] = [
6265
],
6366
component: ForecastFormatter,
6467
},
65-
{ appName: ['fdi'], component: ForecastDataInventory },
68+
{
69+
appName: ['fdi', 'forecast data inventory'],
70+
component: ForecastDataInventory,
71+
},
6672
{ appName: ['fluid-symphony', 'Fluid Symphony'], component: FluidSymphony },
6773
{ appName: ['orca'], component: Orca },
6874
{
@@ -95,8 +101,11 @@ export const ApplicationIcon = forwardRef<HTMLDivElement, ApplicationIconProps>(
95101
{ name, size = 48, iconOnly = false, withHover = false, grayScale = false },
96102
ref
97103
) => {
104+
const lowerCaseName = name.toLowerCase();
98105
const appData = apps.find((app) =>
99-
app.appName.includes(name.toLowerCase())
106+
app.appName.some(
107+
(appNameToMatch) => appNameToMatch.toLowerCase() === lowerCaseName
108+
)
100109
);
101110

102111
if (appData === undefined)

0 commit comments

Comments
 (0)