← Back to team overview

openlp-core team mailing list archive

  1. openlp-core team
  2. Mailing list archive
  3. Message #09753

Re: [Merge] lp:~orangeshirt/openlp/bibles into lp:openlp

  Thread Previous • Date Previous • • Thread Next 
Review: Needs Fixing
Hi Armin,

If you use log.error, you should be alerting the user to the error. If the problem is not bad enough to show the user a message, use log.warn, as I said in my previous comment.
-- 
https://code.launchpad.net/~orangeshirt/openlp/bibles/+merge/63224
Your team OpenLP Core is subscribed to branch lp:openlp.

References

  Thread Previous • Date Previous • • Thread Next