Code review comment for lp:~trb143/openlp/audit

Revision history for this message
Raoul Snyman (raoul-snyman) wrote :

This looks mostly fine, I just would rather you created a menu item somewhere (like in the Tools menu) with a checkable menu item that says "Auditing enabled". You could also just make that an option in the settings dialog. A blank media manager item is a bit silly. Also, then you could add more options to the Audit config dialog for things like filename formatting, file size, file location, and anything else you want.

review: Needs Fixing

« Back to merge proposal