Closed (fixed)
Project:
Metatag
Version:
7.x-1.x-dev
Component:
Miscellaneous
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
14 Jun 2011 at 03:46 UTC
Updated:
21 Dec 2012 at 01:58 UTC
Jump to comment: Most recent file
Comments
Comment #1
dave reidI'm not too familiar with the context module so it may take someone else to write this code. Leaving as postponed for now.
Comment #2
dropbydrop commentedsub
Comment #3
dave reidI did not use the correct postponed.
Comment #4
damienmckennaTwo related modules:
I think this should be done as a submodule so it can leverage & standardize on the main module's meta tagging, e.g. #1160340: Rewrite as a submodule of Metatags (D7 branch), possibly inviting other authors to join as co-maintainers for their pieces. Thoughts?
Comment #5
damienmckennaUpdated title.
Comment #6
dakku commentedHI Damien,
You have mail :D
Comment #7
tahiticlic commentedNice Damien!
Comment #8
dropbydrop commentedSo since there is new module, is this issue fixed?
Comment #9
damienmckenna@dropbydrop: I'd like Metatag have the ability to do this out of the box.
Comment #10
damienmckennaDigging into it I think this needs to be a submodule, but I also think it should be the official mechanism for assigning meta tags to content by path rather than by entity.
Comment #11
damienmckennaDrawing a blank. I need to talk with Dave about how to combine the context-driven meta tags with the ones generated by the system. Argh.
Comment #12
damienmckennaFYI I've uploaded some WIP code to the 'context' branch.
Comment #13
Miszel commentedMy integration code can be checked out from here: http://drupal.org/sandbox/marcin_pajdzik/1810914
Comment #14
damienmckennaMarcin Pajdzik++
:)
Comment #15
damienmckennaI've tested out the new sandbox and, well, it's *great*! *Really* good work, Marcin!
Comment #16
blairski commentedI did a test run and it works well. Good job .
Just one thing: change the module name to
. Currently the git clone creates
Comment #17
Miszel commentedThanks for the reviews guys. I have slightly changed the way the metatag "per path" admin page works as the previous solution was unreliable:
#1814074: Improve the level of control over what is being displayed on the metatag per path admin page.
Comment #18
damienmckennaI've merged in Marcin's sandbox into the new 1187710-context branch. I've made a few small changes, inc renaming the admin page's path, but the kudos go to Marcin.
Comment #19
damienmckennaInstructions for checking out the new branch: http://drupal.org/node/640498/git-instructions/1187710-context/nonmainta...
Comment #20
damienmckennaI've made a number of improvements to the Context submodule:
This appears to be working fine.
Comment #21
Miszel commentedOne more fix. Please change:
to:
in metatag_context.admin.inc line 11
Comment #22
damienmckennaCommitted! Thanks everyone for your help with this, especially Marcin!
Comment #23
damienmckennaA little follow-up to update the main module's README.txt file.
Comment #24
damienmckennaI've committed the patch from #23 too.
Comment #25
damienmckennaLast night saw the release of 7.x-1.0-beta1, so I'm closing all these "fixed" issues in the interest of tidying up the issue queue. Thank you all for your help getting us to this point!
Comment #26
remkovdz commentedThis works great, thanks a lot!
Comment #27
mxtOnly to inform you guys: I've installed latest DEV version of context (2012-Oct-10) and no metatada reaction is available in context UI.
Downgrading to context 7.x-3.0-beta4 meradata reaction return available.
Comment #28
damienmckenna@MXT: Please open a new issue for this. Thanks.
Comment #29
mstrelan commented@MXT - see #1871798: Context reaction is not available after module installation until the cache is cleared. This might result in an additional issue for metatag.