openlp-core team mailing list archive
-
openlp-core team
-
Mailing list archive
-
Message #15818
Re: [Merge] lp:~raoul-snyman/openlp/special-song-importer into lp:openlp
oops, continued...
* ... doesn't necessarily mean they're antagonistic towards OpenLP
* This importer isn't 'dodgy'. It's only available to authenticated
users, and gives them exactly the same result as if they manually
downloaded the txt file. Openlp doesn't share copyrighted material
with unlicensed users.
* They don't serve adds on their website, so they won't be losing ad
revenue if users stay in OpenLP instead of going to the website.
* It's kind of tricky to hide something like this, especially in an
open source project. Once the feature is released, users are sure to
eventually talk about it explicitly on the forum and blogs.
Just my two cents.
On 5/31/12, Samuel Findlay <samuel.findlay@xxxxxxxxx> wrote:
> Perhaps a discussion over whether and why this needs to be 'hidden'?
> * Negotiations over the API may have failed but this doesn't
>
> On 5/31/12, Jonathan Corwin <j@xxxxxxxxxxxx> wrote:
>> Review: Needs Fixing
>>
>> Still not entirely sold about hiding it in clear sight on the Tools menu.
>> In
>> my opinion it is no less hidden than on the import menu. If we're not
>> sure
>> we should be doing this then it shouldn't be there at all. If we're going
>> to
>> include it, let's do it properly and put the option where users expect to
>> find it. (Even if a separate option on the import option if it doesn't
>> currently fit in the wizard)
>>
>> Note we now have two imports with the same name. Perhaps the one in the
>> import wizard should be renamed to "CCLI SongSelect text file", or
>> something?
>>
>> 781: Blank line
>>
>> 1342: Do we need to shout?
>> --
>> https://code.launchpad.net/~raoul-snyman/openlp/special-song-importer/+merge/107558
>> You are subscribed to branch lp:openlp.
>>
>
> --
> Sent from my mobile device
>
--
Sent from my mobile device
https://code.launchpad.net/~raoul-snyman/openlp/special-song-importer/+merge/107558
Your team OpenLP Core is subscribed to branch lp:openlp.
References