openlp-core team mailing list archive

[Raoul Snyman <raoul@xxxxxxxxxxx>]

Review: Needs Fixing

Hi Ian,

Thanks for the merge proposal and welcome to the OpenLP dev team!

Three things:

1. Please take a moment to read through our wiki http://wiki.openlp.org/Development:Getting_Started (if you haven't yet)
2. Please write a test or three to check you've fixed the problem properly
3. Please run all the tests against https://ci.openlp.io/ and add the results here

Thanks!
-- 
https://code.launchpad.net/~ieknight1101/openlp/pjlink/+merge/281821
Your team OpenLP Core is subscribed to branch lp:openlp.