Skip to content

chore: update Comms CLI icon#17

Merged
amix merged 1 commit into
mainfrom
amix/update-comms-cli-icon
Jun 5, 2026
Merged

chore: update Comms CLI icon#17
amix merged 1 commit into
mainfrom
amix/update-comms-cli-icon

Conversation

@amix
Copy link
Copy Markdown
Member

@amix amix commented Jun 5, 2026

Context

The checked-in Comms CLI icon still used the old Twist-style artwork in local assets, OAuth registration metadata, and browser auth pages.

What was changed

  • Replaced the SVG/PNG icon assets with the new Comms CLI icon.
  • Updated Dynamic Client Registration to use the current repo icon URL for new OAuth clients.
  • Reused the new SVG in auth success/error pages and added metadata coverage for logoUri.

@amix amix requested a review from scottlovegrove June 5, 2026 10:15
@amix amix marked this pull request as ready for review June 5, 2026 10:15
@amix amix merged commit 7609b47 into main Jun 5, 2026
7 checks passed
@amix amix deleted the amix/update-comms-cli-icon branch June 5, 2026 10:15
@amix amix added the 👀 Show PR PR must be reviewed before or after merging label Jun 5, 2026
Copy link
Copy Markdown
Member

@doistbot doistbot left a comment

Choose a reason for hiding this comment

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

Thanks Amir for smoothly updating the Comms CLI icon assets and OAuth registration metadata; no issues were flagged during the review 😄 🙌.

Share FeedbackReview Logs

Copy link
Copy Markdown
Collaborator

@scottlovegrove scottlovegrove left a comment

Choose a reason for hiding this comment

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

👍🏻

@doist-release-bot
Copy link
Copy Markdown
Contributor

🎉 This PR is included in version 1.5.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

released 👀 Show PR PR must be reviewed before or after merging

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants