-
-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
chore(create): update the facebook template with current assets #7639
Conversation
✅ [V2]
To edit notification comments on pull requests, go to your Netlify site settings. |
⚡️ Lighthouse report for the deploy preview of this PR
|
@@ -1,5 +1,5 @@ | |||
/** | |||
* Copyright (c) Facebook, Inc. and its affiliates. | |||
* Copyright (c) Meta Platforms, Inc. and affiliates. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Are we going to update the copyright headers for the rest of the codebase?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ya we should if we can. I did that with Flux recently
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Shouldn't you also change the Facebook Open-Source logo on that Flux site?
This PR didn't change the logo on our own site, just the template one
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We can, but I wanted to focus on the template since that's used to start new projects. Our position has been that it's not strictly necessary to go update existing projects (and it gets really noisy) but new projects should start with "Meta".
Are we blocked on facebookincubator/infima#260? Or can we merge this without fixing that first |
This is not too urgent. If Infima fixes it and the site will inherit the fix automatically, then we can merge first. |
alt: 'Meta Open Source Logo', | ||
// This default includes a positive & negative version, allowing for | ||
// appropriate use depending on your site's style. | ||
src: 'img/meta_opensource_logo_negative.png', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this is a svg now, will fix it
This uses the new logo, updates copyrights, etc.
Note: It currently looks suboptimal because of facebookincubator/infima#260