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

Aleix Pol Gonzalez aleixpol at kde.org
Thu Apr 7 11:14:07 UTC 2016


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


Fix it, then Ship it!




Thanks!


plasmoid/package/contents/ui/Sftp.qml (line 46)
<https://git.reviewboard.kde.org/r/127583/#comment64164>

    Remove such comments.


- Aleix Pol Gonzalez


On April 7, 2016, 8:47 a.m., David Kahles wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/127583/
> -----------------------------------------------------------
> 
> (Updated April 7, 2016, 8:47 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/20160407/1d821e0e/attachment.html>


More information about the KDEConnect mailing list