It has the potential to lock up the entire plasma workspace session

[xiangzhai]

I'm a bit concerned by the blocking calls in AccountsService here:
https://github.com/hawaii-desktop/qtaccountsservice/blob/master/src/accountsservice/accountsmanager.cpp#L107

That has the potential to lock up the entire plasma workspace session; which obviously isn't ideal.
Blocking calls are a problem if we can't rely on the remote service /always/ behaving.

[plfiorini]

Yep the API needs to be completely async

[xiangzhai]

let`s fix it ;P

[plfiorini]

Yes, added to the API review task list.

The plan:

0.7.0 is a release which should be as much reliable as we can.
This way issues that can lead to API changes are still allowed until 1.0 the most important concerns are already addressed with 0.7.0. With 0.7.0 we can add QtAccountsService as a (still optional) dep on sddm.

This means that 0.7.0 needs unit tests (already started doing that).