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 Aldi #10445

Closed
wants to merge 2 commits into from
Closed

Add Aldi #10445

wants to merge 2 commits into from

Conversation

Garbg
Copy link
Contributor

@Garbg Garbg commented Feb 11, 2024

aldi

Issue: closes #10330

Similarweb rank (for US website):
Global: 3,681
Country (US): 681
Category: 5

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

This company has several websites for different countries, but I used the stats from www.aldi.us

@github-actions github-actions bot added the new icon Issues or pull requests for adding a new icon label Feb 11, 2024
@PeterShaggyNoble PeterShaggyNoble changed the title Add global aldi sud Add Aldi Feb 12, 2024
@PeterShaggyNoble
Copy link
Member

Nice find with the trademark guidelines, @Garbg 👍🏻 Unfortunately, though, the line below means we're going to have to seek their permission to add it. Would you like to see if you can make contact with them?

The trademarks, service marks, copyrighted material, patents and other intellectual property owned or used by ALDI Inc. may not be used by you without the express written consent of ALDI Inc.

@PeterShaggyNoble PeterShaggyNoble added the permission required Permission should be sought out from the trademark owner before including this icon in our project. label Feb 13, 2024
@Garbg
Copy link
Contributor Author

Garbg commented Feb 13, 2024

I don't want to request permission because I don't know who to ask, and I know nothing about the law.

@adamrusted
Copy link
Member

I don't know who to ask

Looking at their website, this is probably the best starting point we have.
Could you drop them a message, @Garbg? We can't look to add this icon until we've got permission (previous Aldi icons were added by a former employee of the Schwarz Group).

@dirien dirien mentioned this pull request Apr 7, 2024
@dirien
Copy link
Member

dirien commented Apr 7, 2024

@adamrusted: I don't see anything written for the usage of Aldi (Nord or Süd).

@PratyushJayachandran
Copy link
Member

I don't know who to ask

Looking at their website, this is probably the best starting point we have. Could you drop them a message, @Garbg? We can't look to add this icon until we've got permission (previous Aldi icons were added by a former employee of the Schwarz Group).

Looking at Aldi corporate website where the logo is hosted, the terms of use says that:

Except with the express written consent of ALDI Inc., the materials in this site may be used only for personal, non-commercial purposes.

Refer: https://corporate.aldi.us/newsroom/multimedia-library/terms-of-use/

So, aren't we good to add this @adamrusted?

@adamrusted
Copy link
Member

So, aren't we good to add this @adamrusted?

Nope, we can't guarantee our users will use only for personal / non-commercial purposes, and the fact still stands that we'd need written permission from them for any other use case.

@Garbg
Copy link
Contributor Author

Garbg commented May 20, 2024

I'll close this for now

@Garbg Garbg closed this May 20, 2024
@adamrusted adamrusted mentioned this pull request May 20, 2024
@adamrusted adamrusted removed the permission required Permission should be sought out from the trademark owner before including this icon in our project. label Jun 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new icon Issues or pull requests for adding a new icon
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Request: Aldi
5 participants