Skip to content

chore(site): add postgres icon #19210

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.

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

Merged
merged 7 commits into from
Aug 13, 2025
Merged

chore(site): add postgres icon #19210

merged 7 commits into from
Aug 13, 2025

Conversation

AJ0070
Copy link
Contributor

@AJ0070 AJ0070 commented Aug 7, 2025

added pgadmin logo as mentioned in review:
image

I have read the CLA Document and I hereby sign the CLA

@AJ0070 AJ0070 requested a review from aslilac as a code owner August 7, 2025 02:39
@cdr-bot cdr-bot bot added the community Pull Requests and issues created by the community. label Aug 7, 2025
Copy link

github-actions bot commented Aug 7, 2025

All contributors have signed the CLA ✍️ ✅
Posted by the CLA Assistant Lite bot.

@aslilac
Copy link
Member

aslilac commented Aug 7, 2025

I can't find where this specific file was sourced from, and we need to make sure we have the proper license to redistribute it. can you share where you found this at?

@DevelopmentCats
Copy link
Contributor

I can't find where this specific file was sourced from, and we need to make sure we have the proper license to redistribute it. can you share where you found this at?

It was sourced from here: https://www.vectorlogo.zone/logos/postgresql

This is the presskit info for postgres: https://www.postgresql.org/about/press/presskit17/#graphics

@aslilac
Copy link
Member

aslilac commented Aug 8, 2025

that does not at all resemble the icon that I am seeing in the diff

aslilac

This comment was marked as outdated.

@aslilac aslilac changed the title Added pgadmin logo chore(site): add postgres logo Aug 11, 2025
@aslilac aslilac changed the title chore(site): add postgres logo chore(site): add postgres icon Aug 11, 2025
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.

just needs cla signature

@DevelopmentCats
Copy link
Contributor

@AJ0070 can you reply with your sign off for the CLA?

@AJ0070
Copy link
Contributor Author

AJ0070 commented Aug 12, 2025

@AJ0070 can you reply with your sign off for the CLA?

how can I sign it? I added it in the PR description

@aslilac
Copy link
Member

aslilac commented Aug 12, 2025

sorry, the bot only checks comments, not the description. and it checks for exact text.

@AJ0070
Copy link
Contributor Author

AJ0070 commented Aug 12, 2025

I have read the CLA Document and I hereby sign the CLA

cdrci2 added a commit to coder/cla that referenced this pull request Aug 12, 2025
@AJ0070
Copy link
Contributor Author

AJ0070 commented Aug 12, 2025

sorry, the bot only checks comments, not the description. and it checks for exact text.

sorry I didn't know that

@matifali
Copy link
Member

/cherry-pick release/2.24

@matifali
Copy link
Member

/cherry-pick release/2.25

@DevelopmentCats DevelopmentCats merged commit a556324 into coder:main Aug 13, 2025
27 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Aug 13, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
community Pull Requests and issues created by the community.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants