<table><tr><td style="">dvratil updated the summary for this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-6lhahkuvgr5pjqz/" rel="noreferrer">(Show Details)</a><br />dvratil updated this revision to Diff 7869.<br />dvratil added a comment.
</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/D3140" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>Change wording a bit (see updated summary), removed dead and unused code</p></div></div><br /><div><strong>EDIT DETAILS</strong><div><div style="white-space: pre-wrap; color: #74777D;"><div style="padding: 8px 0;">...</div>The application/pgp-keys formatter simply prints a message saying that this is an OpenPGP key and explains what that is, as seen below:<br />
<span style="padding: 0 2px; color: #802b2b; background: rgba(251, 175, 175, .7);">{F377010}</span><span style="padding: 0 2px; color: #3e6d35; background: rgba(151, 234, 151, .6);">{F396575}</span><br />
<br />
User can click the "Show key details" link to get details about the key (uid, creation date, fingerprint), and if the key is available in local keyring there will also be "Open in Kleopatra" link.<br />
<span style="padding: 0 2px; color: #802b2b; background: rgba(251, 175, 175, .7);">{F377011</span><span style="padding: 0 2px; color: #3e6d35; background: rgba(151, 234, 151, .6);">{F396576</span>}<br />
<br />
<br />
<br />
The application/vnd.gnupg.<span style="padding: 0 2px; color: #802b2b; background: rgba(251, 175, 175, .7);">wks mimetype formatter is more complicated. It handles WKS publication confirmation requests. The formatter prints a message saying that this is a Key publication confirmation request and explains what it is. It shows a button in the middle saying "Checking key state..." and checks the WKD for the key. If the key is found, it says that the key has been published and allows no further interaction.<br />
{F377012}<br />
<br />
<br />
If it does not find the key in the WKD, it show "Publish key" button.<br />
{F377013}<br />
<br />
 When clicked, it will create the confirmation response and send it to the WKD server and update the view again. It uses SmtpJob, so there's no email left behind in "Sent" folder.<br />
{F377014}<br />
<br />
After the response is successfully sent we go an extra mile and we store the state in an Akonadi::Attribute in</span><span style="padding: 0 2px; color: #3e6d35; background: rgba(151, 234, 151, .6);">wks mimetype formatter handles</span> the <span style="padding: 0 2px; color: #802b2b; background: rgba(251, 175, 175, .7);">respective Akonadi::Item for the message</span><span style="padding: 0 2px; color: #3e6d35; background: rgba(151, 234, 151, .6);">WKS Publishing Confirmation request</span>. <span style="padding: 0 2px; color: #802b2b; background: rgba(251, 175, 175, .7);">We then check for the presence of th</span><span style="padding: 0 2px; color: #3e6d35; background: rgba(151, 234, 151, .6);">The request </span>is <span style="padding: 0 2px; color: #802b2b; background: rgba(251, 175, 175, .7);">attribute</span><span style="padding: 0 2px; color: #3e6d35; background: rgba(151, 234, 151, .6);">worded</span> in <span style="padding: 0 2px; color: #802b2b; background: rgba(251, 175, 175, .7);">case the user would open the email again before the server manages to process </span>the <span style="padding: 0 2px; color: #802b2b; background: rgba(251, 175, 175, .7);">confirmation response and publish the key</span><span style="padding: 0 2px; color: #3e6d35; background: rgba(151, 234, 151, .6);">terms of "Registration"</span>, which <span style="padding: 0 2px; color: #802b2b; background: rgba(251, 175, 175, .7);">would result in KMail again showing</span><span style="padding: 0 2px; color: #3e6d35; background: rgba(151, 234, 151, .6);">might be better for</span> the <span style="padding: 0 2px; color: #802b2b; background: rgba(251, 175, 175, .7);">"Publish key" button (despite response already being sent).</span><span style="padding: 0 2px; color: #3e6d35; background: rgba(151, 234, 151, .6);">user,</span> <span style="padding: 0 2px; color: #802b2b; background: rgba(251, 175, 175, .7);">Instead when the attribute is present we show the "Key already published" message.<br />
<br />
I would appreciate your feedback mostly regarding the user-facing strings, which can all be found in the .html files</span><span style="padding: 0 2px; color: #3e6d35; background: rgba(151, 234, 151, .6);">and has a button to "Register".<br />
{F396579}<br />
<br />
Once clicked, the formatter silently sends the confirmation response to WKS and deletes both the request and the response</span>.<br />
</div></div></div><br /><div><strong>REPOSITORY</strong><div><div>rKDEPIMADDONS KDE PIM Addons</div></div></div><br /><div><strong>CHANGES SINCE LAST UPDATE</strong><div><a href="https://phabricator.kde.org/D3140?vs=7628&id=7869" rel="noreferrer">https://phabricator.kde.org/D3140?vs=7628&id=7869</a></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D3140" rel="noreferrer">https://phabricator.kde.org/D3140</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>CMakeLists.txt<br />
kdepim-addons.categories<br />
plugins/messageviewer/bodypartformatter/CMakeLists.txt<br />
plugins/messageviewer/bodypartformatter/gnupgwks/CMakeLists.txt<br />
plugins/messageviewer/bodypartformatter/gnupgwks/Messages.sh<br />
plugins/messageviewer/bodypartformatter/gnupgwks/application_gnupgwks.desktop<br />
plugins/messageviewer/bodypartformatter/gnupgwks/gnupgwksformatter.cpp<br />
plugins/messageviewer/bodypartformatter/gnupgwks/gnupgwksformatter.h<br />
plugins/messageviewer/bodypartformatter/gnupgwks/gnupgwksmessagepart.cpp<br />
plugins/messageviewer/bodypartformatter/gnupgwks/gnupgwksmessagepart.h<br />
plugins/messageviewer/bodypartformatter/gnupgwks/gnupgwksurlhandler.cpp<br />
plugins/messageviewer/bodypartformatter/gnupgwks/gnupgwksurlhandler.h<br />
plugins/messageviewer/bodypartformatter/gnupgwks/pgpkeyformatter.cpp<br />
plugins/messageviewer/bodypartformatter/gnupgwks/pgpkeyformatter.h<br />
plugins/messageviewer/bodypartformatter/gnupgwks/pgpkeymemento.cpp<br />
plugins/messageviewer/bodypartformatter/gnupgwks/pgpkeymemento.h<br />
plugins/messageviewer/bodypartformatter/gnupgwks/pgpkeymessagepart.cpp<br />
plugins/messageviewer/bodypartformatter/gnupgwks/pgpkeymessagepart.h<br />
plugins/messageviewer/bodypartformatter/gnupgwks/pgpkeyurlhandler.cpp<br />
plugins/messageviewer/bodypartformatter/gnupgwks/pgpkeyurlhandler.h<br />
plugins/messageviewer/bodypartformatter/gnupgwks/plugin.cpp<br />
plugins/messageviewer/bodypartformatter/gnupgwks/plugin.h<br />
plugins/messageviewer/bodypartformatter/gnupgwks/templates.qrc<br />
plugins/messageviewer/bodypartformatter/gnupgwks/templates/gnupgwksmessagepart.html<br />
plugins/messageviewer/bodypartformatter/gnupgwks/templates/pgpkeymessagepart.html</div></div></div><br /><div><strong>EMAIL PREFERENCES</strong><div><a href="https://phabricator.kde.org/settings/panel/emailpreferences/" rel="noreferrer">https://phabricator.kde.org/settings/panel/emailpreferences/</a></div></div><br /><div><strong>To: </strong>dvratil, bjoernbalazs, mlaurent, aheinecke<br /><strong>Cc: </strong>knauss, emanuel, mlaurent, kde-pim, KDE PIM, spencerb, dvasin, winterz, vkrause, dvratil<br /></div>