← Back to team overview

openlp-core team mailing list archive

Re: [Merge] lp:~tomasgroth/openlp/remote-sync into lp:openlp

 

Some initial comments:

 - Put the "backends" folder at the same level as the "lib" folder
 - I'd put the "synchronizer" module on the same level as backends since it contains base classes, and only put the actual backends in backends
 - Get rid of the "lib" folder if you can
 - Also, I think this should be in core
-- 
https://code.launchpad.net/~tomasgroth/openlp/remote-sync/+merge/336573
Your team OpenLP Core is requested to review the proposed merge of lp:~tomasgroth/openlp/remote-sync into lp:openlp.


References