nicolasfella added inline comments.

INLINE COMMENTS

> kdeplatformdependent.cpp:165
> +    // directly requests the accounts kcm to start adding a new account if 
> it's not there, maybe even pre-fills the fields...)
> +    KService::List services = 
> KServiceTypeTrader::self()->query(QStringLiteral("KCModule"), 
> QStringLiteral("Library == 'kcm_kaccounts'"));
> +    // If we failed to get the kcm, tell the caller we failed

This sounds like a use case for KCMultiDialog, like e.g. in 
https://invent.kde.org/kde/kdeconnect-kde/-/blob/master/settings/main.cpp

REPOSITORY
  R119 Plasma Desktop

REVISION DETAIL
  https://phabricator.kde.org/D25961

To: leinir
Cc: nicolasfella, apol, zachus, plasma-devel, Orage, LeGast00n, 
The-Feren-OS-Dev, cblack, jraleigh, fbampaloukas, GB_2, ragreen, ZrenBot, 
ngraham, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, 
ahiemstra, mart

Reply via email to