Make it possible to modify vcards

Authored by apol on Aug 6 2019, 2:08 PM.



Offers the raw vcard property and allows getting a new raw vcard and refresh from it.
Depends on D22968

Test Plan

Modified my contacts using plasma-phonebook

Diff Detail

R732 KPeople VCard Support
Automatic diff as part of commit; lint not applicable.
Automatic diff as part of commit; unit tests not applicable.
apol requested review of this revision.Aug 6 2019, 2:08 PM
apol created this revision.
apol updated this revision to Diff 64316.Aug 22 2019, 3:40 PM

Make it possible to create new contacts

bshah added inline comments.Aug 23 2019, 4:26 PM

Shouldn't we potentially use the same location? Any specific reason to use different location?

apol added inline comments.Aug 23 2019, 5:31 PM

I don't really know, does it matter? If you think it's better I'm fine with it.

At the moment, kdeconnect-sms already puts it into a subdirectory per device.

jbbgameich accepted this revision.Aug 29 2019, 3:59 PM

Seems fine to me. The vcard location probably doesn't really matter after all, and we can even change it later if necessary.

This revision is now accepted and ready to land.Aug 29 2019, 3:59 PM
This revision was automatically updated to reflect the committed changes.