← Back to team overview

openlp-core team mailing list archive

Re: [Merge] lp:~j-corwin/openlp/general into lp:openlp

 

Review: Needs Fixing
708	+ searchPlugins: function (event) {

Does this method actually perform the search, or just return a list of searchable plugins? If the latter, please rename it to something like "getSearchablePlugins".

799	+ def plugin(self, action):

Possibly rename this one too?
-- 
https://code.launchpad.net/~j-corwin/openlp/general/+merge/61047
Your team OpenLP Core is subscribed to branch lp:openlp.


References