Closed (fixed)
Project:
Omniture Integration
Version:
6.x-2.x-dev
Component:
Miscellaneous
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
12 Aug 2009 at 16:06 UTC
Updated:
1 Sep 2009 at 16:20 UTC
Jump to comment: Most recent file
Comments
Comment #1
ultimateboy commentedI have approved the CVS application. #547310: cytefx [cytefx]
Comment #2
gregglesAnd now a co-maintainer.
In terms of how we want to operate on this module in general:
1. Post an issue as soon as you know you want to make the change.
2. Post a patch here before committing it
3. Wait somewhere between 1 minute and 1 week to get feedback depending on the size of the change (small changes that were agreed to by other maintainers after step 1 should get committed quickly, bigger changes where nobody has commented should wait a little).
We also have a bit of a weird situation with CVS (maybe it's OK).
http://cvs.drupal.org/viewvc.py/drupal/contributions/modules/omniture/om... shows that the omniture.module revision 1.6 is the branch point for both 6.x-2.x and 6.x-1.x ? Maybe we can just work in the DRUPAL-6--2 branch and everything will be "ok".
Comment #3
Race.it commentedThe first Patch for the issue http://drupal.org/node/552318, posted here as per instructions.
Comment #4
Race.it commentedupdated patch to have the correct year in copyright.
Comment #5
gregglesBeing co-maintainer is "fixed" ;)