Skip to content
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

ClickHouse destination doesn't have a logo #19283 #19715

Merged
merged 1 commit into from
Nov 29, 2022

Conversation

juweins
Copy link
Contributor

@juweins juweins commented Nov 22, 2022

What

Neither the dropdown at source nor destination menu showed the clickhouse icon.

How

  1. Corrected typo in destination_definitions.yaml
  2. Added respective icon to source_definitions.yaml

Recommended reading order

straight forward

🚨 User Impact 🚨

minimal

Pre-merge Checklist

No checklist relevant.

  • First Contributor. (And proud!)

Tests

Docker container has to be updated in order to check with local installation. Should work

@CLAassistant
Copy link

CLAassistant commented Nov 22, 2022

CLA assistant check
All committers have signed the CLA.

@marcosmarxm
Copy link
Member

Hello 👋, first thank you for this amazing contribution.

We really appreciate the effort you've made to improve the project.
We ask you patience for the code review. Last month our team was focused on Hacktoberfest event and that probably left some PR without the proper feedback. And this week, due to the Thanksgiving US Holiday, most our team is out of office with their families. Another important piece of information why code won't be merge this week is: as a safety measure the core team has decided to freeze merging code to main branch to keep the release stable. Next week we'll return to you with the proper code review and update the status of your contribution.

If you have any questions feel free to send me a message in Slack!
Thanks!

@juweins
Copy link
Contributor Author

juweins commented Nov 23, 2022

No worries. In the meantime:

Can you provide me with some infos on how & where to get these svg files? (legally correct)

I would like to contribute some from time to time!

Copy link
Member

@marcosmarxm marcosmarxm left a comment

Choose a reason for hiding this comment

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

Thanks!

@marcosmarxm
Copy link
Member

The icon was there: https://github.com/airbytehq/airbyte/blob/master/airbyte-config/init/src/main/resources/icons/clickhouse.svg but only a naming problem.
Usually you try to get in the provider website

@marcosmarxm marcosmarxm merged commit 208f2ec into airbytehq:master Nov 29, 2022
@juweins
Copy link
Contributor Author

juweins commented Nov 29, 2022

The icon was there: https://github.com/airbytehq/airbyte/blob/master/airbyte-config/init/src/main/resources/icons/clickhouse.svg but only a naming problem. Usually you try to get in the provider website

Is there any standard procedure to request them from the desired company? Or is it OK to just download it from the website/html code on my own? (Aiming at copyright related issues)

Thanks for the merge! :)

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

Successfully merging this pull request may close these issues.

4 participants