App Links is the standard Facebook is pushing and using for deep-linking into mobile apps. For content embedded on Facebook, this is the best way to have links directly to an app.

This adds a metatag_al module that implements all current tags.

Comments

deviantintegral’s picture

Status: Active » Needs review
StatusFileSize
new7.2 KB

Tada! This is a format-patch so if there are any followups they are easily diff'ed.

dave reid’s picture

Yay for adding this support. It might be better to use 'app_links' instead of 'al' in the module name and internal data structures? This would follow the pattern established from the existing metatag_opengraph and metatag_twitter_card submodules (yes I realize that metatag_dc does this too but I would say that module violates naming conventions too).

deviantintegral’s picture

I like to lean towards shorter names since some of the hooks can get unwieldly, but I don't feel *that* strongly about it.

damienmckenna’s picture

How about "metatag_applinks"? And yes, metatag_dc should have been named differently :-\

damienmckenna’s picture

Oh, while the website is named "applinks.org", the site itself specifically states "app links" in pretty much every occasion. So yeah, "metatag_app_links" would be best. And yeah, I guess even metatag_opengraph should be named "metatag_open_graph". Meh. #hindsight.

deviantintegral’s picture

StatusFileSize
new7.42 KB

Renamed!

damienmckenna’s picture

StatusFileSize
new8.54 KB

I've made some minor tweaks and added a README.txt file.

damienmckenna’s picture

Status: Needs review » Fixed

Committed.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.