Skip to content

[RELEASE] fix: branded icon name matching#3313

Merged
drewstone merged 1 commit into
developfrom
fix/branded-icon-matching
Jun 25, 2026
Merged

[RELEASE] fix: branded icon name matching#3313
drewstone merged 1 commit into
developfrom
fix/branded-icon-matching

Conversation

@drewstone

Copy link
Copy Markdown
Contributor

Normalize spaces to hyphens before matching blueprint names to branded SVG icons.

@drewstone drewstone requested a review from AtelyPham as a code owner June 25, 2026 13:30
tangletools
tangletools previously approved these changes Jun 25, 2026

@tangletools tangletools left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

✅ Auto-approved PR — c9f0e8bb

Blanket team auto-approval is enabled for this reviewer service.
The full PR reviewer audit still runs separately and will publish findings if it detects issues.

tangletools · auto-approval · reason: blanket_auto_approve · 2026-06-25T13:30:39Z

Blueprint names have spaces (AI Trading) but map keys have hyphens
(ai-trading). Normalize both to hyphen-separated before matching.
@drewstone drewstone dismissed tangletools’s stale review June 25, 2026 13:34

The merge-base changed after approval.

@drewstone drewstone force-pushed the fix/branded-icon-matching branch from c9f0e8b to 3baa556 Compare June 25, 2026 13:34

@tangletools tangletools left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

✅ Auto-approved PR — 3baa5561

Blanket team auto-approval is enabled for this reviewer service.
The full PR reviewer audit still runs separately and will publish findings if it detects issues.

tangletools · auto-approval · reason: blanket_auto_approve · 2026-06-25T13:34:30Z

@drewstone drewstone merged commit 39f5a2b into develop Jun 25, 2026
8 checks passed
@drewstone drewstone deleted the fix/branded-icon-matching branch June 25, 2026 13:34
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