Skip to content

fix(azure): use Azure icon for Azure DevOps connector and remove AzureDevOpsIcon#5118

Merged
waleedlatif1 merged 1 commit into
stagingfrom
fix/icon-azure
Jun 17, 2026
Merged

fix(azure): use Azure icon for Azure DevOps connector and remove AzureDevOpsIcon#5118
waleedlatif1 merged 1 commit into
stagingfrom
fix/icon-azure

Conversation

@waleedlatif1

Copy link
Copy Markdown
Collaborator

Summary

  • Replaced the Azure DevOps icon with the Azure icon on the Azure DevOps knowledge base connector
  • Removed AzureDevOpsIcon entirely; repointed the three Azure DevOps triggers to AzureIcon
  • Updated the icon mock in connectors/utils.test.ts

Type of Change

  • Bug fix

Testing

Tested manually

Checklist

  • Code follows project style guidelines
  • Self-reviewed my changes
  • Tests added/updated and passing
  • No new warnings introduced
  • I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

@vercel

vercel Bot commented Jun 17, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
docs Skipped Skipped Jun 17, 2026 8:36pm

Request Review

@cursor

cursor Bot commented Jun 17, 2026

Copy link
Copy Markdown

PR Summary

Low Risk
UI-only icon swap with no auth, sync, or trigger behavior changes.

Overview
Standardizes Azure DevOps branding in Sim by dropping the dedicated AzureDevOpsIcon SVG from icons.tsx and wiring the Azure DevOps knowledge connector plus all three Azure DevOps triggers to the shared AzureIcon.

The connector utils test icon mock is updated so imports still resolve after the rename.

Reviewed by Cursor Bugbot for commit 7802934. Configure here.

@waleedlatif1 waleedlatif1 merged commit fcfa41c into staging Jun 17, 2026
11 checks passed
@waleedlatif1 waleedlatif1 deleted the fix/icon-azure branch June 17, 2026 20:37
@greptile-apps

greptile-apps Bot commented Jun 17, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This PR replaces the dedicated AzureDevOpsIcon component with the generic AzureIcon across all Azure DevOps integration points (connector meta, three trigger configs, and their test mock), then deletes the now-unused icon component from apps/sim/components/icons.tsx.

  • Icon removal: AzureDevOpsIcon (~30 lines including gradient SVG) is deleted from the sim icons file; all four consumers are updated to import AzureIcon instead.
  • Test update: The vi.mock in connectors/utils.test.ts is updated to stub AzureIcon rather than AzureDevOpsIcon, keeping unit tests aligned with the new import.
  • Note: apps/docs/components/icons.tsx retains its own copy of AzureDevOpsIcon (defined but not consumed by anything in docs); that file is out of scope for this PR but is a leftover to clean up separately.

Confidence Score: 5/5

Safe to merge — purely a cosmetic icon swap with no logic, API, or data changes.

All four consumers of AzureDevOpsIcon in the sim app are updated consistently, the test mock is kept in sync, and the deleted component has no remaining references in apps/sim. The only loose end is the orphaned AzureDevOpsIcon definition in apps/docs/components/icons.tsx, which is a pre-existing condition unaffected by this PR.

No files require special attention. apps/docs/components/icons.tsx retains a now-unused AzureDevOpsIcon definition, but it is outside this PR's scope.

Important Files Changed

Filename Overview
apps/sim/components/icons.tsx Removes the AzureDevOpsIcon component (~30 lines), which is no longer referenced anywhere in the sim app.
apps/sim/connectors/azure-devops/meta.ts Swaps the connector icon from AzureDevOpsIcon to AzureIcon; no logic changes.
apps/sim/connectors/utils.test.ts Updates the vi.mock for @/components/icons to replace AzureDevOpsIcon with AzureIcon, keeping the test in sync.
apps/sim/triggers/azure_devops/build_failed.ts Swaps trigger icon import from AzureDevOpsIcon to AzureIcon; no logic changes.
apps/sim/triggers/azure_devops/webhook.ts Swaps trigger icon import from AzureDevOpsIcon to AzureIcon; no logic changes.
apps/sim/triggers/azure_devops/work_item_created.ts Swaps trigger icon import from AzureDevOpsIcon to AzureIcon; no logic changes.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[AzureIcon\napps/sim/components/icons.tsx] --> B[azure-devops/meta.ts\nConnector]
    A --> C[azure_devops/build_failed.ts\nTrigger]
    A --> D[azure_devops/webhook.ts\nTrigger]
    A --> E[azure_devops/work_item_created.ts\nTrigger]
    F["~~AzureDevOpsIcon~~ (removed)"] -. was used by .-> B
    F -. was used by .-> C
    F -. was used by .-> D
    F -. was used by .-> E
Loading
%%{init: {'theme': 'base', 'themeVariables': {"darkMode": true, "background": "#0d1117", "primaryColor": "#21262d", "primaryTextColor": "#e6edf3", "primaryBorderColor": "#8b949e", "lineColor": "#8b949e", "textColor": "#e6edf3", "edgeLabelBackground": "#161b22", "actorBkg": "#21262d", "actorBorder": "#8b949e", "actorTextColor": "#e6edf3", "actorLineColor": "#8b949e", "signalColor": "#8b949e", "signalTextColor": "#e6edf3", "noteBkgColor": "#373320", "noteBorderColor": "#d4a72c", "noteTextColor": "#f0e6c0", "labelBoxBkgColor": "#21262d", "labelBoxBorderColor": "#8b949e", "labelTextColor": "#e6edf3", "loopTextColor": "#e6edf3", "activationBkgColor": "#30363d", "activationBorderColor": "#8b949e"}}}%%
flowchart TD
    A[AzureIcon\napps/sim/components/icons.tsx] --> B[azure-devops/meta.ts\nConnector]
    A --> C[azure_devops/build_failed.ts\nTrigger]
    A --> D[azure_devops/webhook.ts\nTrigger]
    A --> E[azure_devops/work_item_created.ts\nTrigger]
    F["~~AzureDevOpsIcon~~ (removed)"] -. was used by .-> B
    F -. was used by .-> C
    F -. was used by .-> D
    F -. was used by .-> E
Loading

Reviews (1): Last reviewed commit: "fix(azure): replace Azure DevOps icon wi..." | Re-trigger Greptile

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.

1 participant