Filing a request for a 5.0 upgrade.

CommentFileSizeAuthor
#1 mailhandler_d5.patch17.47 KBChrisKennedy

Comments

ChrisKennedy’s picture

Status: Active » Needs review
StatusFileSize
new17.47 KB

Here is some initial work to port mailhandler to d5. I haven't tested much functionality but it's a start.

moshe weitzman’s picture

Thanks ChrisK ... I'm looking for help with development and testing here. I'm not actively working on this module right now.

ChrisKennedy’s picture

Sorry, I'm not sure I understand what your comment meant. What do you think about applying the patch to HEAD and/or a DRUPAL-5 branch so that the issue queue can be used to resolve any remaining bugs?

moshe weitzman’s picture

ChrisK - thats a good idea. I'd like for *someone* to have tested this thoroughly though. When you say "i haven't tested much", I get worried. So just test some more, and I will commit. If you want, I am happy to give you access to this project so you can help maintain.

sangamreddi’s picture

Thanks Chris, the patch seems to be working fine for me. I'll test for some time and update the output.

teradome’s picture

Does this mean that HEAD can be tested under 5.0? Or has the patch not been committed yet? I too am seriously missing mailhandler in the new Drupal...

ChrisKennedy’s picture

Okay with this patch I was able to add an account, delete an account, edit an account, and retrieve mail.

moshe weitzman’s picture

Status: Needs review » Reviewed & tested by the community

ok, thats sufficient reviewing for me. i am setting to rtbc and adding ChrisKennedy as a committer. i am not really lookig after this module at the moment.

ChrisKennedy’s picture

Status: Reviewed & tested by the community » Fixed
Anonymous’s picture

Status: Fixed » Closed (fixed)