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

Open
wants to merge 7 commits into
base: main
Choose a base branch
from
Open

chore(site): add postgres icon #19210

wants to merge 7 commits into from

Conversation

AJ0070
Copy link

@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
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
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
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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
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