Skip to content

fix(v2): correct typo in metas generated for Twitter cards#3900

Merged
slorber merged 1 commit intofacebook:masterfrom
mpsq:mp-twitter
Dec 10, 2020
Merged

fix(v2): correct typo in metas generated for Twitter cards#3900
slorber merged 1 commit intofacebook:masterfrom
mpsq:mp-twitter

Conversation

@mpsq
Copy link
Copy Markdown
Contributor

@mpsq mpsq commented Dec 9, 2020

Motivation

The Open Graph protocol format is the following:

<meta property="og:image" content="https://example.com/image.png />

Twitter metas, however, are structured in a different format:

<meta name="twitter:image" content="https://example.com/image.png />

They are similar but not identical, I think this led to confusion and
the typo we have in the codebase.

The Open Graph protocol format is the following:

```
<meta property="og:image" content="https://example.com/image.png />
```

Twitter metas, however, are structured in a different format:

```
<meta name="twitter:image" content="https://example.com/image.png />
```

They are similar but not identical, I think this led to confusion and
the typo we have in the codebase.
@mpsq mpsq requested review from lex111 and slorber as code owners December 9, 2020 15:55
@facebook-github-bot facebook-github-bot added the CLA Signed Signed Facebook CLA label Dec 9, 2020
@netlify
Copy link
Copy Markdown

netlify bot commented Dec 9, 2020

✔️ Deploy preview for docusaurus-2 ready!

🔨 Explore the source changes: ca5722c

🔍 Inspect the deploy logs: https://app.netlify.com/sites/docusaurus-2/deploys/5fd2455b6307fd144317b87f

😎 Browse the preview: https://deploy-preview-3900--docusaurus-2.netlify.app

@github-actions
Copy link
Copy Markdown

github-actions bot commented Dec 9, 2020

Size Change: 0 B

Total Size: 154 kB

ℹ️ View Unchanged
Filename Size Change
website/build/blog/2017/12/14/introducing-docusaurus/index.html 20.7 kB 0 B
website/build/docs/introduction/index.html 180 B 0 B
website/build/index.html 5.82 kB 0 B
website/build/main.********.js 109 kB 0 B
website/build/styles.********.css 17.5 kB 0 B

compressed-size-action

@slorber slorber merged commit 656c04e into facebook:master Dec 10, 2020
@slorber
Copy link
Copy Markdown
Collaborator

slorber commented Dec 10, 2020

Thanks

@markerikson FYI that might impact your customization on Redux sites?

@slorber slorber added the pr: bug fix This PR fixes a bug in a past release. label Dec 10, 2020
@mpsq mpsq deleted the mp-twitter branch December 10, 2020 16:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed Signed Facebook CLA pr: bug fix This PR fixes a bug in a past release.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants