Skip to content

Commit b83999f

Browse files
Fix missing integration icons in the local sidebar (#1715)
Co-authored-by: Rhys Sullivan <39114868+RhysSullivan@users.noreply.github.com>
1 parent 349b049 commit b83999f

1 file changed

Lines changed: 10 additions & 2 deletions

File tree

‎packages/app/src/web/shell.tsx‎

Lines changed: 10 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,10 @@ import {
99
toolsAllAtom,
1010
} from "@executor-js/react/api/atoms";
1111
import { Button } from "@executor-js/react/components/button";
12-
import { integrationPresetIconUrl } from "@executor-js/react/components/integration-favicon";
12+
import {
13+
integrationInferredUrl,
14+
integrationPresetIconUrl,
15+
} from "@executor-js/react/components/integration-favicon";
1316
import { IntegrationIconWithAccount } from "@executor-js/react/components/integration-icon-with-account";
1417
import { CommandPalette } from "@executor-js/react/components/command-palette";
1518
import { useClientPlugins, useIntegrationPlugins } from "@executor-js/sdk/client";
@@ -135,10 +138,15 @@ function IntegrationList(props: { pathname: string; onNavigate?: () => void }) {
135138
>
136139
<IntegrationIconWithAccount
137140
icon={integrationPresetIconUrl(
138-
{ id: slug, kind: integration.kind },
141+
{ id: slug, kind: integration.kind, name, url: integration.displayUrl },
139142
integrationPlugins,
140143
)}
141144
integrationId={slug}
145+
url={
146+
integration.displayUrl ??
147+
integrationInferredUrl({ id: slug, name }) ??
148+
undefined
149+
}
142150
size="sm"
143151
/>
144152
<span className="flex-1 truncate">{name}</span>

0 commit comments

Comments
 (0)