Review Request: Enable file transfer channel creation from contact list

Martin Klapetek martin.klapetek at gmail.com
Mon May 2 00:50:04 CEST 2011



> On May 1, 2011, 10:44 p.m., Daniele Elmo Domenichelli wrote:
> > main-widget.cpp, lines 464-465
> > <http://git.reviewboard.kde.org/r/101263/diff/1/?file=15707#file15707line464>
> >
> >     This is a copy-paste from lines 445-446. :D
> >     Should I modify it before committing or leave it as it is?

No need, these lines will get modified once the groupping is merged back.


- Martin


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


On May 1, 2011, 6:03 p.m., Daniele Elmo Domenichelli wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/101263/
> -----------------------------------------------------------
> 
> (Updated May 1, 2011, 6:03 p.m.)
> 
> 
> Review request for Telepathy and Martin Klapetek.
> 
> 
> Summary
> -------
> 
> Initial implementation of file transfer channel creation from the contact list
> 
> Probably will require a fix in tp-qt4 because the URI property is not correctly set (only the filename is set), therefore the handler ignores the path the file.
> 
> 
> Diffs
> -----
> 
>   main-widget.h cfcd045201fc357093f837e3aa5f0c2d4e371d00 
>   main-widget.cpp 0781d4194aefaff679441dc43276e5765c18e19e 
> 
> Diff: http://git.reviewboard.kde.org/r/101263/diff
> 
> 
> Testing
> -------
> 
> The channel is created and received by the file transfer handler
> 
> 
> Thanks,
> 
> Daniele Elmo
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/kde-telepathy/attachments/20110501/5c0b768c/attachment.htm 


More information about the KDE-Telepathy mailing list