Add open on remote device (Desktop)
ClosedPublic

Authored by nicolasfella on Sep 5 2018, 2:00 PM.

Details

Summary

Add openFile to Share Plugin and extend handler to open local file urls

Future extension: Modify the desktop file to allow Open with > Open on connected device

Depends on D16605

Test Plan

Apply Android patch. Use kdeconnect-handler file:///somefile. Check phone for reaction

Diff Detail

Repository
R224 KDE Connect
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
nicolasfella created this revision.Sep 5 2018, 2:00 PM
Restricted Application added a project: KDE Connect. · View Herald TranscriptSep 5 2018, 2:00 PM
Restricted Application added a subscriber: kdeconnect. · View Herald Transcript
nicolasfella requested review of this revision.Sep 5 2018, 2:00 PM

Could you expand on the test plan for this? As in, what should I see?
Currently (on a build from the master branch) if I do kdeconnect-handler file://<file>, it downloads the file to my phone and the app spawns a notification giving me the option to open it. What does this patch change?

Could you expand on the test plan for this? As in, what should I see?
Currently (on a build from the master branch) if I do kdeconnect-handler file://<file>, it downloads the file to my phone and the app spawns a notification giving me the option to open it. What does this patch change?

Sorry, I am dumb and it didn't connect to me that you had an android-side and desktop-side patch

I am unable to get the devices to connect. The desktop complains with 'kdeconnect.core: Failing due to ("The certificate is self-signed, and untrusted")'

Would it be possible to rebase these on master and re-upload?

  • Default to file sharing again, add option to open
albertvaka accepted this revision.Nov 4 2018, 10:44 AM
This revision is now accepted and ready to land.Nov 4 2018, 10:44 AM
This revision was automatically updated to reflect the committed changes.