<table><tr><td style="">nowicki added inline comments.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D7775" rel="noreferrer">View Revision</a></tr></table><br /><div><strong>INLINE COMMENTS</strong><div><div style="margin: 6px 0 12px 0;"><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D7775#inline-33784" rel="noreferrer">View Inline</a><span style="color: #4b4d51; font-weight: bold;">dvratil</span> wrote in <span style="color: #4b4d51; font-weight: bold;">ewsfetchcalendardetailjob.cpp:122</span></div>
<div style="margin: 8px 0; padding: 0 12px; color: #74777D;"><p style="padding: 0; margin: 8px;">This looks unused now?</p>

<p style="padding: 0; margin: 8px;">Does the EWSItem contain actual information regarding the timezone, except for its name, like UTC offset and DST transitions?</p>

<p style="padding: 0; margin: 8px;">I know realize KCalCore may not have the API you might need to get this working properly, but I don't know how much information you actually have available for EWS events</p></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">You're right - I missed that when converting to QDateTime.</p>

<p style="padding: 0; margin: 8px;">In the current form the data downloaded from Exchange contains full vCalendar payload including the timezone specification. This I believe is handled by the updated KCalCore code, that I asked about in one of the previous comments.</p>

<p style="padding: 0; margin: 8px;">This implementation is however temporary as it doesn't provide room to implement calendar modifications. To do that I'll need to download the events piece by piece including the timezone. The Exchange server provides the timezone name as well as offset information, so it will be enough to do proper conversion. This is the moment I'll need to hook up to KCalCore's timezone processing code, but I'm not too worried about it ATM.</p></div></div><br /><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D7775#inline-33786" rel="noreferrer">View Inline</a><span style="color: #4b4d51; font-weight: bold;">dvratil</span> wrote in <span style="color: #4b4d51; font-weight: bold;">ewsfetchcalendardetailjob.cpp:177</span></div>
<div style="margin: 8px 0; padding: 0 12px; color: #74777D;"><p style="padding: 0; margin: 8px;">If the errorText of the job is passed to Akonadi::ResourceBase::cancelTask() at some point, it should be localized (and probably less technical, that's what debug output is for)</p></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">This used to be like that in the beginning, but after addressing Laurent's comments I have changed this behaviour - the exact message is only printed to the log and the user is presented with a more generic message that item retrieval has failed (see ewsresource.cpp:608)</p></div></div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R44 KDE PIM Runtime</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D7775" rel="noreferrer">https://phabricator.kde.org/D7775</a></div></div><br /><div><strong>To: </strong>nowicki, KDE PIM, dvratil, mlaurent<br /><strong>Cc: </strong>kde-pim, dvasin, winterz, vkrause, mlaurent, knauss, dvratil<br /></div>