Closed (fixed)
Project:
Metatag
Version:
7.x-1.x-dev
Component:
Other tags
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
18 Feb 2015 at 20:54 UTC
Updated:
20 Apr 2015 at 14:54 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
deviantintegral commentedTada! This is a format-patch so if there are any followups they are easily diff'ed.
Comment #2
dave reidYay 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).
Comment #3
deviantintegral commentedI like to lean towards shorter names since some of the hooks can get unwieldly, but I don't feel *that* strongly about it.
Comment #4
damienmckennaHow about "metatag_applinks"? And yes, metatag_dc should have been named differently :-\
Comment #5
damienmckennaOh, 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.
Comment #6
deviantintegral commentedRenamed!
Comment #7
damienmckennaI've made some minor tweaks and added a README.txt file.
Comment #8
damienmckennaCommitted.