telephony-service

Merge lp://qastaging/~phablet-team/telephony-service/online-account-match into lp://qastaging/telephony-service/staging

  1. online-account-match
  2. Merge into staging
Proposed by Renato Araujo Oliveira Filho
Status: Merged
Approved by: Gustavo Pichorim Boiko
Approved revision: 1246
Merged at revision: 1250
Proposed branch: lp://qastaging/~phablet-team/telephony-service/online-account-match
Merge into: lp://qastaging/telephony-service/staging
Prerequisite: lp://qastaging/~phablet-team/telephony-service/qml-start-chat
Diff against target: 100 lines (+43/-14)
2 files modified
libtelephonyservice/contactwatcher.cpp (+41/-14)
libtelephonyservice/contactwatcher.h (+2/-0)
To merge this branch: bzr merge lp://qastaging/~phablet-team/telephony-service/online-account-match
Reviewer Review Type Date Requested Status
Gustavo Pichorim Boiko (community) Approve
Review via email: mp+318022@code.qastaging.launchpad.net

Commit message

Implemented contact match by online accounts (IRC).

To post a comment you must log in.
lp://qastaging/~phablet-team/telephony-service/online-account-match updated
1246. By Tiago Salem Herrmann

merge parent branch

Revision history for this message
Gustavo Pichorim Boiko (boiko) wrote :

I think the correct solution for this would be to use the addressableFields each account provides, but for now this implementation will do.

Can you just add an entry to the TODO list saying that maybe the X-IRC match should be done by providing the correct vcard fields in the messaging-framework plugin?

review: Needs Fixing
Revision history for this message
Renato Araujo Oliveira Filho (renatofilho) wrote :

> I think the correct solution for this would be to use the addressableFields
> each account provides, but for now this implementation will do.
>
> Can you just add an entry to the TODO list saying that maybe the X-IRC match
> should be done by providing the correct vcard fields in the messaging-
> framework plugin?

I am not sure if I understand what do you expect from "addressableFields". The contacts are not stored as Vcards and some fields does not have 1 to 1 relationship with the QContactFields, some kind of translation is inevitable.

Revision history for this message
Gustavo Pichorim Boiko (boiko) wrote :

> > I think the correct solution for this would be to use the addressableFields
> > each account provides, but for now this implementation will do.
> >
> > Can you just add an entry to the TODO list saying that maybe the X-IRC match
> > should be done by providing the correct vcard fields in the messaging-
> > framework plugin?
>
> I am not sure if I understand what do you expect from "addressableFields". The
> contacts are not stored as Vcards and some fields does not have 1 to 1
> relationship with the QContactFields, some kind of translation is inevitable.

For the IRC case, I would expect the addressableFields reported by the connection to contain the value "x-irc", and then use that to generate the contact filters. Doing a translation is fine, as long as we use the fields that the account reports as the reference for that.

Revision history for this message
Gustavo Pichorim Boiko (boiko) wrote :

The code looks good, so approving it.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches

to all changes: