Review: Approve

Great merge proposal.

One small thing, unless you're specifically looking for an empty string, you 
can generally just type "if string:" instead of "if string != '':" and likewise 
for the reverse.
-- 
https://code.launchpad.net/~stewart-e/openlp/bug-1310084/+merge/216527
Your team OpenLP Core is subscribed to branch lp:openlp.

_______________________________________________
Mailing list: https://launchpad.net/~openlp-core
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~openlp-core
More help   : https://help.launchpad.net/ListHelp

Reply via email to