Hello, I converted the feedfield module to Drupal 6.x. I'm not exactly sure what's the normal way of contributing something like this, so I just zipped it and am going to upload it with this post.

It seems to work fine, but it could use some more testing and cleaning up (I threw it together fairly quickly). So it should either be labelled as a dev version, or an alpha. There are also some more features I would like to add sometime in the future. If I could get CVS access to this project, that would be sweet :). Or, I could just upload patches later, doesn't matter.

Comments

greggles’s picture

Status: Needs review » Needs work

subscribe (this is not actually a patch)

YaxBalamAhaw’s picture

StatusFileSize
new28.91 KB

Since it's pretty much a new version, I wasn't sure if I should upload a patch or the whole module. Anyway, here's a patch file.

Thomas_Zahreddin’s picture

Category: support » task
Priority: Normal » Critical
Status: Needs work » Needs review

Hi,

YaxBalamAhaw i suggest, you make a formal request to become the maintainer of this projcet, instructions: http://drupal.org/node/251466

Best
Thomas

the greenman’s picture

I am very happy to add more maintainers. I am unfortunately too busy to manage all my modules right now, and would be delieghted to get someone else on board!

YaxBalamAhaw’s picture

Status: Needs review » Patch (to be ported)
StatusFileSize
new45.99 KB

Hello, I've cleaned up some of the code, added some optional field settings, extensively tested everything, and everything seems to be working as it should. The patch file is attached.

YaxBalamAhaw’s picture

StatusFileSize
new46.03 KB

Oops, I diffed the wrong folder. The patched attached to this should be used.

greggles’s picture

@YaxBalamAhaw - at this point I think you would be justified in taking over the module. See http://drupal.org/node/251466 for more details.

the greenman’s picture

Hi YaxBalamAhaw

I would love to add you as a maintainer, I am not going to be able to manage things for a while.
So, if you get me your CVS account name, or apply for one, I'll get you on board.
If you apply for an account, just point to this comment in your application.

A message to CVS gatekeepers - Please give YaxBalamAhaw CVS access so that he can become a maintainer of this module.

Thanks very much for the great work.

avpaderno’s picture

FYI, I approved YaxBalamAhaw's account, and granted him access to this project CVS.

YaxBalamAhaw’s picture

Priority: Critical » Normal
Status: Patch (to be ported) » Fixed

Thanks. I branched the previous HEAD branch into DRUPAL-5, committed my code to HEAD, branched it to DRUPAL-6--1, then tagged it for release as DRUPAL-6--1-0.

Status: Fixed » Closed (fixed)

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