<table><tr><td style="">leinir created this revision.<br />Herald added a project: Frameworks.<br />Herald added a subscriber: kde-frameworks-devel.<br />leinir requested review of this revision.
</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/D28532">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>Prior to this, we did have error reporting, but only in the way of<br />
writing errors out on the command line (through qCCritical). While<br />
this is fine for debugging purposes, it is unfortunate when errors<br />
can be expected to occur in day to day use (as we are dealing with<br />
a wide range of things that can (and often do) go wrong), and giving<br />
proper feedback to the user as to why, for example, their new icons<br />
aren't showing up when they seem to have installed just fine would<br />
seem a reasonable thing to do.</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">Add a signal to Installation fired when there's a critical error</li>
<li class="remarkup-list-item">Forward the installer errors through KNSCore::Engine</li>
<li class="remarkup-list-item">Add the MessageBoxSheet component from kaccounts-integration</li>
<li class="remarkup-list-item">Add a component for displaying errors from the engine</li>
<li class="remarkup-list-item">Use the ErrorDisplayer component (only show on the current page)</li>
</ul></div></div><br /><div><strong>REPOSITORY</strong><div><div>R304 KNewStuff</div></div></div><br /><div><strong>BRANCH</strong><div><div>more-installation-error-handling (branched from master)</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D28532">https://phabricator.kde.org/D28532</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>src/core/engine.cpp<br />
src/core/installation.cpp<br />
src/core/installation.h<br />
src/qtquick/qml/EntryDetails.qml<br />
src/qtquick/qml/Page.qml<br />
src/qtquick/qml/private/EntryCommentsPage.qml<br />
src/qtquick/qml/private/ErrorDisplayer.qml<br />
src/qtquick/qml/private/MessageBoxSheet.qml</div></div></div><br /><div><strong>To: </strong>leinir<br /><strong>Cc: </strong>kde-frameworks-devel, LeGast00n, cblack, GB_2, michaelh, ngraham, bruns<br /></div>