[Applicationjobs Data Engine] Expose job "destUrl"
ClosedPublic

Authored by broulik on Dec 26 2016, 10:00 PM.

Details

Summary

This will eventually allow the notification applet to show a better finished notification.
Currently, it just uses whatever was set as second info label last. While this works for when copying a single file, it's utterly wrong when copying folders as this will be a random file that was copied last.

Test Plan

I can now (still wip local patches I need to clean up a bit)

  • show the destination of a job while it's still figuring out what do copy (e.g. when no info label 2 is set, I'll show dest Url there instead)
  • show Copying finished: /foo/bar instead of Copying finished: /foo/bar/some/random/file/that/was/copied/also.txt (need to figure out so a single file copied/received still shows the full path for easy access)

Depends on D3818

Diff Detail

Repository
R120 Plasma Workspace
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
broulik updated this revision to Diff 9374.Dec 26 2016, 10:00 PM
broulik retitled this revision from to [Applicationjobs Data Engine] Expose job "destUrl".
broulik updated this object.
broulik edited the test plan for this revision. (Show Details)
broulik added a reviewer: Plasma.
broulik set the repository for this revision to R120 Plasma Workspace.
Restricted Application added a project: Plasma. · View Herald TranscriptDec 26 2016, 10:00 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
mart accepted this revision.Dec 27 2016, 11:59 AM
mart added a reviewer: mart.
This revision is now accepted and ready to land.Dec 27 2016, 11:59 AM
This revision was automatically updated to reflect the committed changes.