[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
Lint Skipped
Unit
Unit Tests Skipped
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.