Re: [Evolution-hackers] New plugin (Purge Old Mail) [Patch attached]

[Parthasarathi Susarla <sparthasarathi novell com>]

Hey Alex,
So the plugin looks good an works well too.

A couple of issues:
* you need to probably update the eplug file for the right hooks (as
discussed on IRC)
* in the function org_gnome_purge_mail_prefs_factory, you are comparing
strings (folder names), since folder names are translatable, this check
would fail for folder names which are translated. 

Otherwise, things look fine.

Thanks and all the best for the plugins to come :)

Cheers,
partha
On Tue, 2005-11-08 at 12:16 -0200, Alexandre Rocha Lima e Marcondes
wrote:
> Hello fellow Evolution Hackers,
> 
>     I'm proud to announce a patch to incorporate a new plug-in to
> evolution: purge old mail ....
>     This plug-in allows to set the number of days to keep messages on
> folder-by-folder basis as well as enable/disable the plug-in to any
> specific folder.
> 
>     The plug-in files and the patch that is used to compile and add it
> to evolution code base is attached.
> 
>     P.S.: Thank you very much Shreyas Srinivasan for helping me on
> this first journey to code an evolution plugin ...
> 
> --
> Regards,
>   Alexandre Rocha Lima e Marcondes
>    P4 Tecnologia e Desenvolvimento Humano 
> alexandre p4tecnologia com
> alexandre psl-pr softwarelivre org
> arlm users sourceforge net 
> www.p4tecnologia.com
> alexandre.p4tecnologia.com 
> Projetos:
> MonoBrasil MonoBASIC
> ACBr 
> Freedom ERP 
> 
> 
> 
> 
> 
> 
> To validate this message's signature follow the instructions: http://www.cacert.org/index.php?id=3&lang=en_US
> _______________________________________________
> Evolution-hackers mailing list
> Evolution-hackers gnome org
> http://mail.gnome.org/mailman/listinfo/evolution-hackers