Rather than building your own thing you should work with Dave Reid to improve the Metatags module. He's working on the core architecture right now, once it's up you'll be able to see where the gaps are vs what you're currently planning and could then either write a separate module that extends it or build a sub-module to use Context to define the tags. Otherwise we'll have tremendous amounts of unnecessary duplication between two modules.

Comments

dakku’s picture

dakku CreditAttribution: dakku commented

Thanks,
You are right, there needs to be one consistent, complete module.

I think Dave is in the process of rewriting nodewords/metatags module. Once his code is up, I can re factor my module to support metatags :) It is one of our milestones already.

Dave Reid’s picture

Dave Reid
he/him
Primary language English
Location Nebraska USA
CreditAttribution: Dave Reid commented

The problem is I wanted to build this support from within metatags, not have a separate project, especially if the goal is panels support. Your help with #1151938: Panels integration - meta discussion once the initial API and UI are up is appreciated.

dakku’s picture

dakku CreditAttribution: dakku commented

Hey Dave,

Sounds like a plan, lets compare notes once you have done your initial setup and see how we can work in tandem. Are you planning on implementing a "meta data by URL" feature? Because this is the initial purpose of our module.

Dave Reid’s picture

Dave Reid
he/him
Primary language English
Location Nebraska USA
CreditAttribution: Dave Reid commented

I'm not exactly sure what context it will involve, so feel free to start plugging away at this in your project. Once I have the base up and running I'll take a look at this to see what you have.

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna commented
Version: » 6.x-1.x-dev

So Metatag is D7 only, while Context_Metadata is D6 only. I might be interested in possibly merging some of Context_Metadata into Nodewords to provide an alternative configuration mechanism, but I need to review the code first.

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna commented
Title: Work on integration with Metatags module rather than a separate module » Rewrite as a submodule of Metatags (D7 branch)
Version: 6.x-1.x-dev » 7.x-1.x-dev

Updated the title to better suit the goals. Also related: #1187710: Integrate with Context

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna commented

Marcin Pajdzik turned this into a sandbox: http://drupal.org/sandbox/marcin_pajdzik/1810914

I'm hoping to merge his work into the main module, I think it'll be the best option for per-path meta tags.

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna commented
Status: Active » Fixed

I've tested Marcin's code and it's great! Lets focus on #1187710: Integrate with Context for D7?

Status: Fixed » Closed (fixed)

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