<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 solid;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="http://git.reviewboard.kde.org/r/104363/">http://git.reviewboard.kde.org/r/104363/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On March 22nd, 2012, 2:25 p.m., <b>David Edmundson</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Code looks fine: but;
1)when you create an account in the accounts KCM, we should check this flag and set the autoConnect appropriately. Otherwise new accounts are not affected, and out of sync.
2) eventually it needs to happen on a per account basis and that's where things get /cRaZy/</pre>
</blockquote>
<p>On March 22nd, 2012, 2:27 p.m., <b>David Edmundson</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Edit. This isn't true is it - you have the kded module that will set it each time.
Ignore the above.
</pre>
</blockquote>
</blockquote>
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">The kded sets it on each login for each account.
About per account basis: as for now we don't have a per account presence it doesn't make much sense to have a per account restoration.
If we have per account presences somewhere in the future it needs to be adapted for per account settings.
But for now (0.4) I think this should do it.</pre>
<br />
<p>- Dominik</p>
<br />
<p>On March 22nd, 2012, 2:17 p.m., Dominik Cermak wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.org/media/rb/images/review_request_box_top_bg.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for Telepathy.</div>
<div>By Dominik Cermak.</div>
<p style="color: grey;"><i>Updated March 22, 2012, 2:17 p.m.</i></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">It's a simple approach to have this feature implemented.
It uses the ConnectsAutomatically property and the AutomaticPresence.
In the kcm you can enable/disable this feature (ConnectsAutomatically is set) and on every presence change the new presence is set as the AutomaticPresence.
This way mission-control cares for setting the saved presence as soon as possible.
Note: Disabling it takes only effect after the second login, I suspect something with the change notification between kcm and kded isn't working (that signal kcm should send and kded receive). So kded sets the property only after starting...</pre>
</td>
</tr>
</table>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Bugs: </b>
<a href="http://bugs.kde.org/show_bug.cgi?id=281929">281929</a>
</div>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>CMakeLists.txt <span style="color: grey">(c55fa8431464b62a5426a6cf29319c3f6f5fce6f)</span></li>
<li>autoconnect.h <span style="color: grey">(PRE-CREATION)</span></li>
<li>autoconnect.cpp <span style="color: grey">(PRE-CREATION)</span></li>
<li>config/telepathy-kded-config.cpp <span style="color: grey">(1fb515f56500407b7b221d1d5310237f8d94ca7f)</span></li>
<li>config/telepathy-kded-config.ui <span style="color: grey">(0d616c5503b22f37c0ae4d5a94411d0f97b25d38)</span></li>
<li>telepathy-module.h <span style="color: grey">(05d8c3847587049dcc4c9329a3b4d0b0bfee7d49)</span></li>
<li>telepathy-module.cpp <span style="color: grey">(955ceec3a8fdaacaf24463ac4e721d0a53086d30)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/104363/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>