apol added a comment.
In D21369#480148 <https://phabricator.kde.org/D21369#480148>, @albertvaka wrote: > If I understand correctly, the only reason to use KContacts here is so we can use its PhoneNumber type. We could change it to a QString? It is a string now, the advantage of using the PhoneNumber class is that it has some extra metadata with it (e.g. that you get to know if it's a work or home phone number). REPOSITORY R307 KPeople REVISION DETAIL https://phabricator.kde.org/D21369 To: sredman, apol Cc: albertvaka, kde-frameworks-devel, LeGast00n, michaelh, ngraham, bruns