<table><tr><td style="">nicolasfella created this revision.<br />nicolasfella added reviewers: Frameworks, broulik.<br />Herald added a project: Frameworks.<br />Herald added a subscriber: kde-frameworks-devel.<br />nicolasfella 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/D29336">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>With <a href="https://phabricator.kde.org/D29021" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D29021</a> the fallback notification system is gone and with it the need for differentating naming. Remove galago from the naming since it's outdated and might confuse readers unfamiliar with the code and its history. Also inline a now not needed function.</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>Still get notifications, action invoked signal is correctly sent</p>

<p>Depends on <a href="https://phabricator.kde.org/D29021" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D29021</a></p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R289 KNotifications</div></div></div><br /><div><strong>BRANCH</strong><div><div>nogalago</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D29336">https://phabricator.kde.org/D29336</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>src/imageconverter.h<br />
src/notifybypopup.cpp<br />
src/notifybypopup.h<br />
src/notifybyportal.cpp</div></div></div><br /><div><strong>To: </strong>nicolasfella, Frameworks, broulik<br /><strong>Cc: </strong>kde-frameworks-devel, LeGast00n, cblack, michaelh, ngraham, bruns<br /></div>