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

Remove Babylon.js #9980

Merged

Conversation

PeterShaggyNoble
Copy link
Member

Issue: ref: #6317 & #8737

Description

As permission to include a monochrome version of Bablylon.js' icon was not sought prior to #8737 being merged and we have yet to receive a response to our request for permission to do so, we will need to remove Babylon.js from our collection. If permission is granted prior to the release of v11 then this PR can be closed.

@PeterShaggyNoble PeterShaggyNoble added the breaking change Issues or pull requests that include a breaking change and scheduled for the next major release label Dec 1, 2023
@PeterShaggyNoble PeterShaggyNoble added this to the v11.0.0 milestone Dec 1, 2023
@PeterShaggyNoble PeterShaggyNoble marked this pull request as draft December 1, 2023 15:17
@adamrusted adamrusted modified the milestones: v11.0.0, v12.0.0 Dec 4, 2023
@adamrusted adamrusted modified the milestones: v12.0.0, v11.0.0 Dec 4, 2023
@PeterShaggyNoble PeterShaggyNoble marked this pull request as ready for review December 31, 2023 14:07
@adamrusted adamrusted merged commit dc63ec4 into simple-icons:develop Jan 2, 2024
4 checks passed
@PeterShaggyNoble PeterShaggyNoble deleted the remove/babylondotjs branch January 5, 2024 10:34
mondeja added a commit that referenced this pull request Jan 7, 2024
# New Icons

- Cooler Master (#10209)
- DeepCool (#10204)
- Make (#10165)
- SideQuest (#10202)
- SiYuan (#10187)
- Stagetimer (#10104)
- Victron Energy (#10198)
- Wyze (#10188)

# Updated Icons

- Affinity Designer (#10176)
- Affinity Photo (#10176)
- Affinity Publisher (#10176)
- Biome (#10203)
- Google Chrome (#10116)
- INFINITI (#10118)
- Opel (#10159)
- RTL (#10166)
- smart (#10168)
- STARZ (#10169)
- TELE 5 (#10170)
- Wish (#10171)

# Removed Icons

- Babylon.js (#9980)
- Hulu (#10048)
- Pepsi (#10086)
- Uno (#9973)
@jglamp
Copy link
Contributor

jglamp commented May 22, 2024

Has this ever mentioned to Babylon.JS? If not, I can send an issue to their branding to see if we can get this sorted.

@adamrusted
Copy link
Member

Don't think so, no. We've recently started cracking down on requiring permission, and BabylonJS was one that we previously hadn't obtained permission for. Happy for you to reach out if you like, see the new guidelines if you're unsure.

@jglamp
Copy link
Contributor

jglamp commented May 22, 2024

Alright I reached out. See here.

@RaananW
Copy link

RaananW commented May 22, 2024

Answered in the referenced issue. Thanks for reaching out!

@jglamp jglamp mentioned this pull request May 22, 2024
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking change Issues or pull requests that include a breaking change and scheduled for the next major release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants