> On 2009-03-15 18:26:03, Matt Rogers wrote:
> > why do you need this? doesn't skype create Kopete::Contact objects? it 
> > should use those instead.
> 
> Pali Rohár wrote:
>     It is for simple invite people to chat:
>     connect( Kopete::UI::ContactActio, SIGNAL(triggered(const QString &, 
> bool)), this, SLOT(inviteContact(const QString &)) )
>     I can use slot ChatSession::inviteContact with contact id (not create new 
> slot with Kopete::Contact object)
> 
> Matt Rogers wrote:
>     Are you saying that ChatSession::inviteContact doesn't take a 
> Kopete::Contact object?

In docs 
(http://api.kde.org/4.x-api/kdenetwork-apidocs/kopete/libkopete/html/classKopete_1_1ChatSession.html)
 is ChatSession::inviteContact with QString not Kopete::Contact. So if I 
connect SIGNAL triggered(Kopete::Contact*) next I can't use SLOT 
inviteContact(QString).


- Pali


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/243/#review492
-----------------------------------------------------------


On 2009-03-05 22:53:18, Pali Rohár wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/243/
> -----------------------------------------------------------
> 
> (Updated 2009-03-05 22:53:18)
> 
> 
> Review request for Kopete.
> 
> 
> Summary
> -------
> 
> This patch add signal triggered(QString id) in KopeteContactAction (id is 
> string name of contact).
> With this patch is easy implement invite button in skype protocol.
> 
> 
> Diffs
> -----
> 
>   /trunk/KDE/kdenetwork/kopete/libkopete/ui/kopetecontactaction.h 933527 
>   /trunk/KDE/kdenetwork/kopete/libkopete/ui/kopetecontactaction.cpp 933527 
> 
> Diff: http://reviewboard.kde.org/r/243/diff
> 
> 
> Testing
> -------
> 
> I'm using this patch for invite button in skype protocol, and all works.
> 
> 
> Thanks,
> 
> Pali
> 
>

_______________________________________________
kopete-devel mailing list
kopete-devel@kde.org
https://mail.kde.org/mailman/listinfo/kopete-devel

Reply via email to