<div dir="ltr"><div><div>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>------<br><br>Does the JSON blob in Job::setData() match the X-Purpose-OutboundArguments of that plugin?<br><br>I don't seem to be able to access what those args are from C++ like Configuration::neededArguments<br><br>Why JSON here rather than QVariantMap? It's all internal to C++ API.<br><br>---------<br><br>In the pastebin plugin why are you registering the KJob with the job tracker?<br><br>You're returning the progress via the kjob the plugin gives back, and the relevant UI should do it.<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>--------<br><br>saveas/saveasplugin.json<br><br>probably best to avoid swearing in a release.<br><br>--------<br><br>That ktp plugin is fine for a demo of things, but it is not release material.<br><br>The job will return that you've shared things even if you click cancel, also you say it will return a URL in the json blob and it doesn't.<br><br>-----<br><br></div>There's a comment<br><br>     * This shouldn't require to have the job actually running on the same process as the app.<br><br></div><div>I don't get what that means.<br></div><br>​</div>