Greg,

Following the CVS applying form, you have to agree that I'll be accept to create a CVS account to help maintain this module, or even assume it, since it appears that it has not been updated recently and now I have time to take care of it, as we will be using the module.

Please, answer here with an agreement.

Thanks!

CommentFileSizeAuthor
#4 omniture-552318.patch2.51 KBRace.it
#3 omniture-552318.patch2.51 KBRace.it

Comments

ultimateboy’s picture

Status: Active » Fixed

I have approved the CVS application. #547310: cytefx [cytefx]

greggles’s picture

And 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".

Race.it’s picture

StatusFileSize
new2.51 KB

The first Patch for the issue http://drupal.org/node/552318, posted here as per instructions.

Race.it’s picture

Status: Fixed » Needs review
StatusFileSize
new2.51 KB

updated patch to have the correct year in copyright.

greggles’s picture

Status: Needs review » Fixed

Being co-maintainer is "fixed" ;)

Status: Fixed » Closed (fixed)

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