Page MenuHomePhabricator
Differential D18781

Include OpenGraph prefix material in <html> tag if OpenGraph is enabled
ClosedPublic
Actions

Authored by epriestley on Nov 22 2017, 7:27 PM.
Tags
None
Referenced Files
F18834778: D18781.id45068.diff
Sun, Oct 26, 10:45 AM
F18817323: D18781.id45070.diff
Tue, Oct 21, 2:32 PM
F18788048: D18781.id45069.diff
Wed, Oct 15, 2:37 AM
F18787552: D18781.id.diff
Wed, Oct 15, 12:15 AM
F18786795: D18781.diff
Tue, Oct 14, 8:34 PM
F18772779: D18781.id.diff
Thu, Oct 9, 4:41 AM
F18769976: D18781.id.diff
Wed, Oct 8, 10:11 AM
F18764288: D18781.diff
Tue, Oct 7, 5:19 AM
View All Files
Subscribers
None

Details

Reviewers
amckinley
Maniphest Tasks
T13018: Support Discourse's specific dialect of OpenGraph metadata tags to enrich Discourse "OneBox" previews
Commits
rPc3d6c4b0ee8e: Include OpenGraph prefix material in <html> tag if OpenGraph is enabled
Summary

Ref T13018. Discourse doesn't seem to be picking this up yet (see https://discourse.phabricator-community.org/t/opengraph-test-topic/762) so maybe it really needs this meta-meta-XML stuff?

Test Plan

Will push.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Revision Contents
Changeset List

PathSize
resources/
celerity/
1 line
src/
view/
page/
9 lines
2 lines
webroot/
rsrc/
favicons/

Diff 45070

View Options

resources/celerity/map.php

Loading...
View Options

src/view/page/AphrontPageView.php

Loading...
View Options

src/view/page/PhabricatorStandardPageView.php

Loading...
View Options

webroot/rsrc/favicons/opengraph-144x144.png

Loading...
Phabricator · Built by Phacility