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

Added Twitter logo, and rebalanced their order #15

Closed
wants to merge 2 commits into from

Conversation

propensive
Copy link
Contributor

Review by @heathermiller. Not sure if the Twitter and MS logos need to
be approved by the organizations before we use them, but let me know if
you'd like me to chase up on getting that approval.

Review by @heathermiller. Not sure if the Twitter and MS logos need to
be approved by the organizations before we use them, but let me know if
you'd like me to chase up on getting that approval.
@heathermiller
Copy link
Member

heathermiller commented Jan 19, 2017

Hey, thank you very much for proposing this! ...Though indeed you're right that the logos aren't up because we need written permission from both companies to include them. We just finished Twitter's membership paperwork this week, so the next step is to get that written permission to include the logo. Though Morgan Stanley decided to avoid the whole hassle with getting permission to post their logo, because it's so much legwork. We're allowed to mention them in the minutes as far as I understand, but we're just not allowed to post their logo due to all of the effort involved in getting approval.

So, long story short, we won't be able to post MS, and we have to wait a bit to post Twitter's. We can update this PR though to only include Twitter's and I can approve and merge as soon as I get written permission. Would that be OK?

@propensive
Copy link
Contributor Author

Yeah, that's no problem! Not surprised about MS, but I didn't realise the Twitter logo was an issue, seeing as it's used without permission everywhere... though I guess never in a "corporate" context!

@SethTisue
Copy link
Member

@heathermiller time to merge?

@heathermiller
Copy link
Member

No, unfortunately. We don't have written permission to post either. We'll never get permission for MS, but Twitter I should start the process for :-/

@heathermiller
Copy link
Member

So, closing not because I don't appreciate this PR but because I can't ever post MS's logo.

@SethTisue
Copy link
Member

@heathermiller but Jon removed MS's logo in the most recent commit

@SethTisue SethTisue changed the title Added Twitter and Morgan Stanley logos, and rebalanced their order Added Twitter logo, and rebalanced their order Apr 12, 2017
@SethTisue SethTisue reopened this Apr 12, 2017
@SethTisue SethTisue closed this Apr 12, 2017
@SethTisue
Copy link
Member

darnit. I misread your comment. sorry

@sjrd sjrd deleted the include-missing-sponsors branch December 6, 2018 10:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants