qxmpp-project / qxmpp

Cross-platform C++ XMPP client and server library
401 stars 197 forks source link

added some probably forgotten promise resolutions #552

Closed TheBluestBird closed 1 year ago

TheBluestBird commented 1 year ago

PR check list:

The changes are insignificant, it doesn't feel like it requires to document them or do some meta work. But if you'll tell - I'm gonna try to cover that case in unit testing

melvo commented 1 year ago

Thanks! It was introduced by b17284ee7d674416e0d11f1699f73fcc606262d4. Please write the commit message starting with an upper case letter and the imperative.

TheBluestBird commented 1 year ago

This way will do?

melvo commented 1 year ago

Yes, thanks!