jgrulich accepted this revision. jgrulich added a comment. This revision is now accepted and ready to land.
I lived in assumption that "link-watchers" is NMVariantMapMap and not NMVariantMapList, that's why I suggested those changes. I'm sorry for that. I'll fix it locally here, you were correct before. The rest looks good. REVISION DETAIL https://phabricator.kde.org/D17342 To: pranavgade, jgrulich Cc: kde-frameworks-devel, michaelh, ngraham, bruns