https://bugs.kde.org/show_bug.cgi?id=401677

Simon Redman <si...@ergotech.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|CONFIRMED                   |RESOLVED
      Latest Commit|                            |https://invent.kde.org/netw
                   |                            |ork/kdeconnect-android/comm
                   |                            |it/876d1fa3db9a7f9a136ae2ca
                   |                            |be38bc81194b6dfd
         Resolution|---                         |FIXED

--- Comment #59 from Simon Redman <si...@ergotech.com> ---
Git commit 876d1fa3db9a7f9a136ae2cabe38bc81194b6dfd by Simon Redman.
Committed on 10/06/2020 at 03:58.
Pushed by sredman into branch 'master'.

[SMS Helper] Use simple=true parameter for fetching the list of all
conversations

## Summary

Use `content://mms-sms/conversations?simple=true` instead of just
`content://mms-sms/conversations` which should work for more devices (and is
reported to work on Samsung devices)

For some reason, this misses exactly one conversation on my phone and throws
back one invalid thread ID -- I suspect this indicates that my message database
is corrupt :shrug:

## Test Plan

### Before:
Lots of devices were not able to retrieve conversations. See [bug
401677](https://bugs.kde.org/show_bug.cgi?id=401677)

### After:
Hopefully all devices are able to retrieve conversations. At minimum, hopefully
there are no regressions :)

M  +42   -17   src/org/kde/kdeconnect/Helpers/SMSHelper.java

https://invent.kde.org/network/kdeconnect-android/commit/876d1fa3db9a7f9a136ae2cabe38bc81194b6dfd

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to