davidedmundson added a comment.
> Drop group stubs; there will be bad UX because two different polkit actions (and thus two dialogs backtoback) would be needed for apply with groups Don't rule out the obvious other option there - changing accountsservice. It already has all the code, just only exposes admin group. Adding more would be relatively doable and a lot tidier. Would need the use case expanding so we can approach with a ratioanle, but let's follow that up on the bug report. For this patch I think you did the right thing, merge as-is, with a tidier base that's easier to expand upon. REPOSITORY R119 Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D28154 To: cblack, #plasma, #vdg, ngraham Cc: crossi, The-Feren-OS-Dev, davidedmundson, broulik, filipf, ngraham, nicolasfella, zzag, plasma-devel, Orage, LeGast00n, cblack, jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart
