Review Request 127583: Introduce QML component to check for plugin availability

[Aleix Pol Gonzalez]

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



A bunch of properties can be made readonly now.

Other than that, looks good to me and can go in.


plasmoid/package/contents/ui/FindMyPhone.qml (line 30)
<https://git.reviewboard.kde.org/r/127583/#comment64149>

    readonly to all 3?


- Aleix Pol Gonzalez


On April 6, 2016, 3 a.m., David Kahles wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/127583/
> -----------------------------------------------------------
> 
> (Updated April 6, 2016, 3 a.m.)
> 
> 
> Review request for kdeconnect.
> 
> 
> Repository: kdeconnect-kde
> 
> 
> Description
> -------
> 
> Remove those checks from three different places and put them into a reusable
> component, which can be used for different plugins.
> 
> 
> Diffs
> -----
> 
>   plasmoid/package/contents/ui/Battery.qml 4538437b9ef0c70c743c673b5256ea95d11f18c3 
>   plasmoid/package/contents/ui/FindMyPhone.qml 8a666b7bcf56ffb1b3bf1823bbf36dedcfd3a969 
>   plasmoid/package/contents/ui/PluginChecker.qml PRE-CREATION 
>   plasmoid/package/contents/ui/Sftp.qml 9ef4f41765d3bd51a4609850ee8f02d7f2eb2bba 
> 
> Diff: https://git.reviewboard.kde.org/r/127583/diff/
> 
> 
> Testing
> -------
> 
> Enabling and disabling the sftp, battery and findmyphone plugin in the KCM works fine.
> 
> 
> Thanks,
> 
> David Kahles
> 
>

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