ngraham added a comment.
Thanks for the patch! For formatting guidelines, please see https://community.kde.org/Infrastructure/Phabricator#Formatting_your_patch In essence, put "BUG: 383764" on its own line in the Summary section, and remove it from the title. Also please put "FIXED-IN: KDE Frameworks 5.44" on its own line, since that's the version of KDE Frameworks that this will make it into. One comment about the code, indicated below. Works great in testing, and I'm excited to get this in. REPOSITORY R241 KIO REVISION DETAIL https://phabricator.kde.org/D10635 To: meven, dfaure Cc: ngraham, #frameworks, michaelh
