Skip to content

Fix Asana integration icon#2106

Merged
arnestrickmann merged 1 commit into
mainfrom
emdash/asana-yjtft
May 19, 2026
Merged

Fix Asana integration icon#2106
arnestrickmann merged 1 commit into
mainfrom
emdash/asana-yjtft

Conversation

@rabanspiegel
Copy link
Copy Markdown
Contributor

@rabanspiegel rabanspiegel commented May 19, 2026

Summary

  • Replace the handcrafted Asana placeholder SVG with the public Simple Icons Asana SVG asset
  • Keep the icon bundled locally and rendered through the existing integration icon pipeline

Tests

  • pnpm run format
  • pnpm run lint
  • pnpm run typecheck
  • pnpm run test

Closes ENG-1327


Note

Low Risk
Low risk: this only swaps a bundled SVG asset and should only affect how the Asana integration icon renders.

Overview
Updates the bundled src/assets/images/Asana.svg, replacing the previous placeholder circles-based SVG with the official Simple Icons Asana glyph (24x24 viewBox, role="img", and <title>), so the integration icon renders correctly.

Reviewed by Cursor Bugbot for commit ce9ca7e. Bugbot is set up for automated code reviews on this repo. Configure here.

Copy link
Copy Markdown
Contributor

@arnestrickmann arnestrickmann left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🙏

Copy link
Copy Markdown
Contributor

@arnestrickmann arnestrickmann left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🙏

@arnestrickmann arnestrickmann merged commit 44d0ed9 into main May 19, 2026
2 checks passed
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