Fix QXmppAccountMigrationManager issues

qxmpp-project / qxmpp

Cross-platform C++ XMPP client and server library

411 stars 197 forks source link

Closed pasnox closed 3 months ago

pasnox commented 4 months ago

We can only request adding roster items one by one

PR check list:

[X] Document your code
[X] Add \since QXmpp 1.X, QXMPP_EXPORT
[X] Update doc/doap.xml
[X] Add unit tests
[X] Format the code: Run clang-format -i src/<edited-file(s)> tests/<edited-file(s)>

lnjX commented 4 months ago

Ah and can you re-apply your commit onto 1.7? So we can do a 1.7.1 release.

pasnox commented 4 months ago

The unit tests still fail, otherwise thanks for the fix :)

Fixed, remaining CI issues seems unrelated to this change.