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.

_______________________________________________
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