Port to json metadata

Authored by nicolasfella on Nov 30 2019, 4:15 PM.

Diff Detail

R150 KDE Telepathy KDE Daemon Module
Automatic diff as part of commit; lint not applicable.
Automatic diff as part of commit; unit tests not applicable.
nicolasfella requested review of this revision.Nov 30 2019, 4:15 PM
nicolasfella created this revision.
ltoscano added inline comments.
3 ↗(On Diff #70623)

I can't comment on the rest of the reivew, but as you are changing this string, I think it would be worth to change the "with KDE" part (it comes from the old definition of KDE). Maybe the entire sentence needs to be re-though, I don't have a clear alternative right now.

Thank you for taking this up.
Would you mind to use the same approach as in plasma-workspace? In case of your changes we'll lose the translations, but with desktop_to_json we won't have such a problem.


  • Use kcoreaddons_desktop_to_json
akulichalexandr accepted this revision.Dec 3 2019, 11:46 PM
This revision is now accepted and ready to land.Dec 3 2019, 11:46 PM
This revision was automatically updated to reflect the committed changes.