D17081: Combine multiple upload jobs into a single KCompositeJob so only 1 notification will be shown

Albert Vaca Cintora noreply at phabricator.kde.org
Thu Nov 29 22:17:18 GMT 2018


albertvaka added a comment.


  In D17081#368360 <https://phabricator.kde.org/D17081#368360>, @eduisters wrote:
  
  > In D17081#368330 <https://phabricator.kde.org/D17081#368330>, @albertvaka wrote:
  >
  > > The only thing, but seems hard to fix is: when there is a transfer in progress and I add another file, the job is updated to say "sending N+1 files", but on Android it still says "receiving N files".
  >
  >
  > On android it will be updated when the next share packet is received (see D17157 <https://phabricator.kde.org/D17157>). I've thought about that but it I don't think there is a way to improve this without a 2-way protocol or introducing a new share packet to update the running transfer info
  
  
  Indeed, it's complicated to fix. It's quite minor anyway.

REPOSITORY
  R224 KDE Connect

BRANCH
  composite_job_for_uploads

REVISION DETAIL
  https://phabricator.kde.org/D17081

To: eduisters, #kde_connect, albertvaka
Cc: albertvaka, apol, nicolasfella, broulik, kdeconnect, varunp, shivanshukantprasad, skymoore, brute4s99, wistak, dvalencia, rmenezes, julioc, Leptopoda, timothyc, jdvr, Danial0_0, johnq, Pitel, adeen-s, SemperPeritus, daniel.z.tg, jeanv, seebauer, bugzy, MayeulC, menasshock, mikesomov, tctara
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdeconnect/attachments/20181129/dc449d60/attachment-0001.html>


More information about the KDEConnect mailing list