KTp/libkpeople: Difference between revisions

From KDE Community Wiki
< KTp
Line 17: Line 17:
Reinstall KTp, and you have metacontacts everywhere.
Reinstall KTp, and you have metacontacts everywhere.


You may also want to install PersonViewer a tool to view all contact information from all sources (http://quickgit.kde.org/?p=scratch%2Fmklapetek%2FPersonViewer.git)
You may also want to install PersonViewer a tool to view all contact information from all sources (http://quickgit.kde.org/?p=person-viewer.git)


==Tasks==
==Tasks==

Revision as of 15:07, 30 July 2013

About

LibKPeople is a contact aggregation library for KDE.

It is *not* a KTp specific library, we are simply the first adopters as well as the developers so the wiki page is here for the time being. This page is about the libkpeople changes needed to make a release of KTp with libkpeople support.

Big discussions about libkpeople should CC the pim mailing list.

Setting up

Install libkpeople (https://projects.kde.org/projects/playground/network/libkpeople) Install ktp-nepomuk-service (https://projects.kde.org/projects/playground/network/telepathy/ktp-nepomuk-service)

start the nepomuk service either by logging out/in or by running "nepomukservicestub nepomuktelepathyservice"

Reinstall KTp, and you have metacontacts everywhere.

You may also want to install PersonViewer a tool to view all contact information from all sources (http://quickgit.kde.org/?p=person-viewer.git)

Tasks

libkpeople

Status Summary Notes Claimed By
DONE Add KPeople namespace 3 votes to 1 on kpeople vs people [mailto: David
 < David

>]

DONE Update Roles as per discussion [mailto:MCK182
 <MCK182

>]

DONE Restore updates on presence change https://git.reviewboard.kde.org/r/110801/ [mailto:Dave
 <Dave

>]

DONE Remove unused / commented out code [mailto:Dave
 <Dave

>]

DONE Add dialog for finding duplicates [mailto:franck
 <franck

>]

DONE Add Dialog to view PersonData (see PersonViewer) in mck182 scratch [mailto:[email protected]
 <[email protected]

>]

IN PROGRESS Add widget to PersonViewer to view associated websites
TODO Port away from QStandardItemModel It's slow and won't be accepted by PIM. PersonItem and ContactItem are fine, we just need to just not inherit from QStandardItem and do model stuff ourselves (apparently)
DONE Rename examples [mailto: Aleix Pol
 < Aleix Pol

>]

IN PROGRESS Fix tests PersonData 80% done. PersonsModel to do [mailto:[email protected]
 <[email protected]

>]

DONE Cache sub-resources in PersonsData Vishesh to implement in Nepomuk Core [mailto: vhanda
 < vhanda

>]

PIM Demos

Status Summary Notes Claimed By

KTp Stuff

Status Summary Notes Claimed By
DONE Move KTp plugins to k-ci-i
DONE Add method to KTp::ContactsModel runtime switch from libkpeople or not.
TODO Add PersonData method to KTp::Contact
TODO KTp Contact has helper methods to get name + avatar for person
TODO KTp ContactFactory can get personData as a feature on load, otherwise use Tp stuff.
TODO Fix the singleton AccountManager problem I don't know how many times I've shouted at people about the problem with using AM as a singleton.. and now we've done it in libkpeople. Brilliant.
DONE Restore Grouping [mailto:David
 <David

>]


Public API Review

Below are a list of issues, please resolve (either by fixing or discussing) then remove from this list

The list of public classes are:

  • PersonsModel
  • PersonsModelFeature
  • AbstractPersonsPlugin
  • BasePersonsDataSource
  • DuplicatesFinder
  • Match
  • MatchesSolver
  • PersonActionsModel
  • PersonsPluginManager
  • PersonData

Anything else in the main lib is private and thus I don't care.

PersonsModel

Why is addContactsToPerson async but removeContactsFromPerson syncronous?

I /think/ if you call removeContactsFromPerson that contact is now not in the model, not inserted as a contact anywhere.

Testing

See git://anongit.kde.org/scratch/mklapetek/PersonViewer for a generic "new addressbook" KTp has kpeople support merged as an optional dependency



git://anongit.kde.org/scratch/vhanda/delete-pimo-people.git is a useful too for wiping all PIMO:Persons useful for testing merging repeatedly.