Code review comment for lp:~therp-nl/server-env-tools/6.1-fetchmail_attach_from_folder

Revision history for this message
Holger Brunn (Therp) (hbrunn) wrote :

Ronald: That's a very good point, thanks. But I think it depends too much on the configured object/field what the state of a message should be to put it into the code. That's why I made it a configuration option for a folder in rev30, with 'received' as default.

Alexandre: Today I did the refactoring you proposed, now I think all your points are covered. Could you have another look?

« Back to merge proposal