← Back to team overview

openlp-core team mailing list archive

Re: [Merge] lp:~sam92/openlp/bug-1695620 into lp:openlp

 

@Tim: As discussed on IRC, could you please review again?
I had to remove the conditions so that the variables are always available to the template, even if they are empty. They were there to make sure that no empty lines were added to the footer. But now this checking is done in the template.

Also wrt the template in its own file, after discussion we came to agreement that it's not worth the hassle (packaging etc). So we keep it just as a config string.
-- 
https://code.launchpad.net/~sam92/openlp/bug-1695620/+merge/363518
Your team OpenLP Core is subscribed to branch lp:openlp.


References