<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Dec 3, 2015 at 6:16 PM, Aleix Pol <span dir="ltr"><<a href="mailto:aleixpol@kde.org" target="_blank">aleixpol@kde.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span class="">On Tue, Dec 1, 2015 at 4:37 PM, David Edmundson<br>
<<a href="mailto:david@davidedmundson.co.uk">david@davidedmundson.co.uk</a>> wrote:<br>
> AlternativesModel doesn't seem like a good name. Alternate to what?<br>
><br>
> ---------<br>
><br>
> /** @returns the job that will perform the share of the specified @p data.*/<br>
><br>
> what data?<br>
<br>
</span>Eh... right. I'll also change the method name.<br>
<span class=""><br>
><br>
> ------<br>
><br>
> Does the JSON blob in Job::setData() match the X-Purpose-OutboundArguments<br>
> of that plugin?<br>
><br>
> I don't seem to be able to access what those args are from C++ like<br>
> Configuration::neededArguments<br>
><br>
> Why JSON here rather than QVariantMap? It's all internal to C++ API.<br>
<br>
</span>X-Purpose-OutboundArguments is specified in the plugin json file.<br>
<span class=""><br>
><br>
> ---------<br>
><br>
> In the pastebin plugin why are you registering the KJob with the job<br>
> tracker?<br>
><br>
> You're returning the progress via the kjob the plugin gives back, and the<br>
> relevant UI should do it.<br>
<br>
</span>Yeah.. maybe it doesn't make sense. I'll just remove it from there now.<br>
<span class=""><br>
><br>
> --------<br>
><br>
> import Ubuntu.OnlineAccounts 0.1 as OA<br>
><br>
> Can we depend on that? Seems weird.<br>
><br>
> Definitely needs mentioning in CMake<br>
<br>
</span>It's mentioned in here: find_package(QMLModule 0.1 COMPONENTS<br>
Ubuntu.OnlineAccounts)<br>
<span class=""><br>
><br>
> --------<br>
><br>
> saveas/saveasplugin.json<br>
><br>
> probably best to avoid swearing in a release.<br>
<br>
</span>Did it make you swear?<br></blockquote><div><br></div><div>I meant that you have:<br> "Description": "Dummy plugin, to test the shit out of the feature" <br><br></div><div>What I like is seeing how the translators chose to deal with that.<br><br><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<span class=""><br>
><br>
> --------<br>
><br>
> That ktp plugin is fine for a demo of things, but it is not release<br>
> material.<br>
><br>
> The job will return that you've shared things even if you click cancel, also<br>
> you say it will return a URL in the json blob and it doesn't.<br>
<br>
</span>Right, could use some love. I'll see if I can get it there or just<br>
disable it for now.<br></blockquote> <br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<span class=""><br>
><br>
> -----<br>
><br>
> There's a comment<br>
><br>
>      * This shouldn't require to have the job actually running on the same<br>
> process as the app.<br>
<br>
</span>It means that the plugin should be able to run in a separate process.<br>
In a KIO kind of way.<br></blockquote><br></div><div class="gmail_quote">Ah as in all data should be serialisable, don't pass pointers to objects or some such?<br></div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<span class=""><font color="#888888"><br>
Aleix<br>
</font></span><div class=""><div class="h5">_______________________________________________<br>
Kde-frameworks-devel mailing list<br>
<a href="mailto:Kde-frameworks-devel@kde.org">Kde-frameworks-devel@kde.org</a><br>
<a href="https://mail.kde.org/mailman/listinfo/kde-frameworks-devel" rel="noreferrer" target="_blank">https://mail.kde.org/mailman/listinfo/kde-frameworks-devel</a><br>
</div></div></blockquote></div><br></div></div>