----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/5141/ -----------------------------------------------------------
Review request for Kopete. Summary ------- This patch adds an 'account' context and possibility to change following notifications for account: kopete_mail, connection_error[empty], connection_lost, server_error[empty], network_problems[empty], buzz_nudge, cannot_connect. ('[empty]' means that this notification doesn't use anywhere) It groups 'msn_mail' and 'yahoo_mail' to one event - 'kopete_mail'. It changes 'icq_authorization' to 'kopete_authorization' (yahoo protocol used this event, but actually it didn't exist) It adds an 'Configure Notifications' button to Accounts KCM, which shows dialog with ability to change notifications for current account. Diffs ----- /trunk/KDE/kdenetwork/kopete/kopete/config/accounts/CMakeLists.txt 1167882 /trunk/KDE/kdenetwork/kopete/kopete/config/accounts/kopeteaccountconfig.h 1167882 /trunk/KDE/kdenetwork/kopete/kopete/config/accounts/kopeteaccountconfig.cpp 1167882 /trunk/KDE/kdenetwork/kopete/kopete/config/accounts/kopeteaccountconfigbase.ui 1167882 /trunk/KDE/kdenetwork/kopete/kopete/kopete.notifyrc 1167882 /trunk/KDE/kdenetwork/kopete/libkopete/kopetechatsession.cpp 1167882 /trunk/KDE/kdenetwork/kopete/libkopete/kopeteutils.cpp 1167882 /trunk/KDE/kdenetwork/kopete/protocols/oscar/icq/icqcontact.cpp 1167882 /trunk/KDE/kdenetwork/kopete/protocols/wlm/wlmaccount.cpp 1167882 /trunk/KDE/kdenetwork/kopete/protocols/yahoo/yahooaccount.cpp 1167882 Diff: http://reviewboard.kde.org/r/5141/diff Testing ------- Kopete from KDE 4.5.0 (and trunk) compiles fine with it. 'Configure Notifications' button works, it shows the dialog and saves all the data. Actually, I can't test buzz_nudge, kopete_mail, connection_lost and cannot_connect (because I don't have yahoo/wlm account, only where it used). I'll try to create an yahoo account and make more testing (or can anyone who have yahoo/wlm account test it?) Thanks, Igor
_______________________________________________ kopete-devel mailing list kopete-devel@kde.org https://mail.kde.org/mailman/listinfo/kopete-devel