Review Request 128939: Implement flashlight flashing in FinMyPhone plugin

[Nicolas Fella]

> On Sept. 19, 2016, 2:40 vorm., Aleix Pol Gonzalez wrote:
> > AndroidManifest.xml, line 30
> > <https://git.reviewboard.kde.org/r/128939/diff/1/?file=477121#file477121line30>
> >
> >     I find it a bit odd to extend KDE Connect permissions to camera for the feature.
> 
> Łukasz Żarnowiecki wrote:
>     If you want to have an access to flashlight, you need to have this permission.  I understand that it might be weird that kind of application wants to have an access to the camera, but this is really a cool feature.  The phone could be much more easily located if you see the strobe light somewhere in the room and if someone is concerned about camera abuse he or she can study the code.  This is really not evil thing.
>     
>     Also the camera permission could be further use to implement other features.
>     
>     * remote shutter
>     * streaming video from camera
>     * using camera in your computer as a normal device
>     * ... and more
> 
> Albert Vaca Cintora wrote:
>     The reason this feature was not included in the first place was because of the camera permission. I don't think it is worth adding an extra permission just for this feature: Adding new permissions makes the Play Store not update the app automatically because it has to prompt the user! which usually results in a notable upgrade funnel. If there are more features (like the ones you mention) that justify adding this permission, then we will of course add it, but as of now I strongly disagree with doing so.

Maybe this could be solved using Android 6's new Permission System (at least for Marshmallow+ devices)


- Nicolas


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/128939/#review99265
-----------------------------------------------------------


On Sept. 18, 2016, 10:19 nachm., Łukasz Żarnowiecki wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/128939/
> -----------------------------------------------------------
> 
> (Updated Sept. 18, 2016, 10:19 nachm.)
> 
> 
> Review request for kdeconnect.
> 
> 
> Repository: kdeconnect-android
> 
> 
> Description
> -------
> 
> I think this a really nice feature when trying to find a phone.
> 
> 
> Diffs
> -----
> 
>   AndroidManifest.xml fdfb80b 
>   src/org/kde/kdeconnect/Plugins/FindMyPhonePlugin/FindMyPhoneActivity.java 2c565bf 
> 
> Diff: https://git.reviewboard.kde.org/r/128939/diff/
> 
> 
> Testing
> -------
> 
> Yes, Android 5.1
> 
> 
> Thanks,
> 
> Łukasz Żarnowiecki
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdeconnect/attachments/20170408/bb1872a5/attachment-0001.html>