----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/101622/#review3935 -----------------------------------------------------------
app/chat-window.cpp <http://git.reviewboard.kde.org/r/101622/#comment3210> I believe that this should be changed to kfiledialog:///FileTransferLastDirectory (same value as contact list, see main-widget.cpp:547) - Daniele Elmo On June 15, 2011, 12:17 a.m., Francesco Nwokeka wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/101622/ > ----------------------------------------------------------- > > (Updated June 15, 2011, 12:17 a.m.) > > > Review request for Telepathy. > > > Summary > ------- > > Patch 2 of 3. > This patch connects the buttons with the corrisponding signals. It also > implements a plasma notification system to notify the user on errors given by > these buttons (took procedure from martin's contactlist). > Buttons implemented are : > ~ video call > ~ audio call > ~ send file > > The invite to chat button will remain disabled (from next patch) until group > support will be ok. After these three patches i will work on that feature > again. > > > Diffs > ----- > > app/CMakeLists.txt dd1a23a > app/chat-window.h 3db916e > app/chat-window.cpp 7fe5f51 > app/telepathy-kde-text-ui.notifyrc PRE-CREATION > > Diff: http://git.reviewboard.kde.org/r/101622/diff > > > Testing > ------- > > Used the buttons to start the different channels and got errors for all apart > for the file transfer because other features are not implemented. Plasma > norification worked well. > > > Thanks, > > Francesco > >
_______________________________________________ KDE-Telepathy mailing list [email protected] https://mail.kde.org/mailman/listinfo/kde-telepathy
