← Back to team overview

openlp-core team mailing list archive

Re: [Merge] lp:~tomasgroth/openlp/dvd into lp:openlp

 

Review: Needs Fixing

Hi,

I tested this again and here are some comments:
- Could you prefill the "Media Path" field with the available devices?
- When I enter a not available path and click "load disc", I cannot change the path again
- When I change the selection in the title combobox, I get the following exception: http://pastie.org/8973290
-- 
https://code.launchpad.net/~tomasgroth/openlp/dvd/+merge/209347
Your team OpenLP Core is requested to review the proposed merge of lp:~tomasgroth/openlp/dvd into lp:openlp.


Follow ups

References