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

Add Alteryx and dbt #5490

Closed
wants to merge 3 commits into from
Closed

Conversation

dsmdavid
Copy link
Contributor

alteryx: #5469
dbt: #5470

Issue: 5469 & 5470
Alexa rank: 26,320 & 84,798

Checklist

  • I updated the JSON data in _data/simple-icons.json
  • I optimized the icon with SVGO or SVGOMG
  • The SVG viewbox is 0 0 24 24

Description

Alteryx icon:
Press release kit: https://www.alteryx.com/company/news/press-kit which links to this google folder: https://drive.google.com/drive/folders/1WIxPiTucm9A97U5qJkZRdKoVOqA6Lv2W

Used the "EPS" "white logo" file in the press release kit.
Hex color taken from the "Alteryx Blue" logo in the press release kit.

dbt icon:
Remove letters from the icon https://github.com/fishtown-analytics/dbt-styleguide/tree/master/_includes/icons and use the hex code present in the styleguide

@github-actions github-actions bot added the new icon Issues or pull requests for adding a new icon label Apr 20, 2021
@PeterShaggyNoble
Copy link
Member

Welcome to Simple Icons, @dsmdavid, and thanks for the contributions 🙂

Before we proceed, though, could you split one of the icons out to a separate PR. This will help speed up the review process as, for example, if there's an issue with one then it's not blocking the other from being merged.

@dsmdavid dsmdavid closed this Apr 20, 2021
@PeterShaggyNoble PeterShaggyNoble added the abandoned Pull requests that have been abandoned by the contributor label Apr 21, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
abandoned Pull requests that have been abandoned by the contributor new icon Issues or pull requests for adding a new icon
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants