Skip to content

chore: add openai icon #19118

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

chore: add openai icon #19118

wants to merge 4 commits into from
+4 −0

Conversation

35C4n0r
Copy link
Contributor

@35C4n0r 35C4n0r commented Aug 1, 2025 •
edited
Loading

@35C4n0r 35C4n0r requested a review from aslilac as a code owner August 1, 2025 14:23
@cdr-bot cdr-bot bot added the community Pull Requests and issues created by the community. label Aug 1, 2025
@35C4n0r 35C4n0r mentioned this pull request Aug 1, 2025
Open
8 tasks
@matifali
Copy link
Member

matifali commented Aug 2, 2025

/cherry-pick release/2.25

@matifali
Copy link
Member

matifali commented Aug 2, 2025

/cherry-pick release/2.24

matifali
matifali requested changes Aug 2, 2025
View reviewed changes
@matifali matifali changed the title feat: add codex icon feat: add open icon Aug 2, 2025
@matifali matifali changed the title feat: add open icon feat: add openai icon Aug 2, 2025
aslilac
aslilac requested changes Aug 3, 2025
View reviewed changes
Copy link
Member

@aslilac aslilac left a comment

Choose a reason for hiding this comment

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

needs an entry in externalImages.ts, which corrects colors based on theme. this one should get the "monochrome" type.

@35C4n0r
Copy link
Contributor Author

35C4n0r commented Aug 3, 2025

Done !

@matifali matifali changed the title feat: add openai icon chore: add openai icon Aug 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@aslilac aslilac aslilac requested changes

@matifali matifali matifali approved these changes

Requested changes must be addressed to merge this pull request.

Assignees

@35C4n0r 35C4n0r

Labels
community Pull Requests and issues created by the community.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@35C4n0r @matifali @aslilac