Skip to content

feat: add CONNECTED_APPS_ENABLED flag and integration icon type map#935

Merged
ajlai-orkes merged 4 commits into
conductor-oss:mainfrom
najeebkp:integrations-enh
Apr 17, 2026
Merged

feat: add CONNECTED_APPS_ENABLED flag and integration icon type map#935
ajlai-orkes merged 4 commits into
conductor-oss:mainfrom
najeebkp:integrations-enh

Conversation

@najeebkp
Copy link
Copy Markdown
Contributor

Pull Request type

  • Bugfix
  • Feature
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • WHOSUSING.md
  • Other (please describe):

NOTE: Please remember to run ./gradlew spotlessApply to fix any format violations.

Changes in this PR

feat: add CONNECTED_APPS_ENABLED flag and resolve integration icons via type map

  • Add CONNECTED_APPS_ENABLED feature flag to flags.ts and CreatorFlags page
  • Add integrationIconMap to resolve integration type identifiers to their
    iconName where they differ (e.g. slack-mcp → slack, openai → openAI)
  • Update IntegrationIcon to pre-resolve via the map before loading SVG,
    falling back to default.svg on error

Comment thread ui-next/src/components/IntegrationIcon.tsx
@ajlai-orkes ajlai-orkes merged commit 82173d2 into conductor-oss:main Apr 17, 2026
6 of 7 checks passed
JamesHarris-orkes pushed a commit to JamesHarris-orkes/conductor that referenced this pull request Apr 27, 2026
…onductor-oss#935)

* feat: add CONNECTED_APPS_ENABLED flag and integration icon type map

* update SidebarMenuTarget type with integrationsSubMenu
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants