1. 21 Jul, 2019 1 commit
  2. 19 Jul, 2019 2 commits
    • Simon Redman's avatar
      [SMS App] Handle addresses for multitarget messages · edee0e0e
      Simon Redman authored
      ## Summary
      Upgrade the SMS App to handle multitarget addresses in the "addresses" field of a message and drop usage of the "address" field
      Also note that this has all the commits from kde/kdeconnect-kde!97, but I will rebase those away once that patch is landed
      Bonus: Image composition for multitarget conversations
      ## Test Plan
      - Apply Android-side patch kde/kdeconnect-android!80
      - Launch SMS App
      - Notice that you can see all the recipients of multitarget messages. (Replying still not supported, but might get implemented as part of fixing replying to single-target messages)
    • Simon Redman's avatar
      [SMS App] Basic plain-text MMS support · ee9547ed
      Simon Redman authored
      ## Summary
      Desktop companion to kde/kdeconnect-android!78
      Give desktop SMS app a basic understanding of the MMSes coming from Android:
      - Show a fake body if we get an attachment we can't display (for now, any attachment)
      - Display a fake contact header for multi-target messages since Android does not yet export multi-target address information
      - Disable attempting to reply to multi-target messages
      BUG: 398889
      ## Test Plan
      ### Before:
      MMS messages were silently dropped, meaning:
       - Group MMS conversations were not visible
       - Single-target conversations with the most-recent message an MMS were not visible
      ### After:
       - Install kde/kdeconnect-android!78
       - Multi-target conversations are displayed (kind of ugly, since they have no contact information
       - Single-target conversations which end with an MMS are displayed
       - Plain-text MMS is displayed nicely
       - MMS attachments don't show
         - MMS which are only an attachment with no body are displayed with a dummy body
  3. 02 Jun, 2019 4 commits
  4. 30 May, 2019 1 commit
  5. 23 Mar, 2019 1 commit
  6. 27 Feb, 2019 1 commit
    • Simon Redman's avatar
      [SMS App] Add highlighting to ConversationList view · 1ec20886
      Simon Redman authored
      Summary: Let the ListView use its default highlighting (did I not notice before?) and keep the currently-selected chat highlighted at all times
      Test Plan:
       - Launch app
       - Use mouse or arrow keys to select a conversation. The highlight should follow the selected conversation
      Reviewers: #kde_connect, apol
      Reviewed By: #kde_connect, apol
      Subscribers: apol, kdeconnect
      Tags: #kde_connect
      Differential Revision: https://phabricator.kde.org/D17612
  7. 11 Oct, 2018 1 commit
  8. 08 Oct, 2018 1 commit
    • Simon Redman's avatar
      [Desktop] Update conversation list when a new message arrives · a7db3ab5
      Simon Redman authored
      When a new message is delivered, the conversation list should update by changing the preview text and re-sorting the conversations
      Bonus bug discovered and fixed:  previously, when the conversations list was being populated, it made a request for the first message in every conversation. This would be fine if the conversationdbusinterface pulled from local cache. However, this actually triggers a request to the phone for *every* conversation.
      This should be handled differently in conversationdbusinterface's requestConversation as well, but that's a project for a later day (TODO comments added)
      Test Plan:
       - Launch SMS app
       - Verify conversations list appears
       - Verify lack of massive stream of debug output indicating lots of messages for the wrong conversation are being received
       - Verify that opening a particular conversation shows the messages after a short delay while the backend fetches the content from the phone
       - Verify that receiving a new message into an existing conversation updates the conversation list
      Reviewers: #kde_connect, nicolasfella
      Reviewed By: #kde_connect, nicolasfella
      Subscribers: nicolasfella, apol, kdeconnect
      Tags: #kde_connect
      Differential Revision: https://phabricator.kde.org/D15608
  9. 07 Oct, 2018 1 commit
    • Simon Redman's avatar
      Correct braces and add contributors · 563dde98
      Simon Redman authored
      Summary: Braces to start a method are on a newline, braces to begin an in-method block are on the same line
      Test Plan: Pure source code cosmetic changes. Hopefully no functionality has changed!
      Reviewers: #kde_connect, nicolasfella
      Reviewed By: #kde_connect, nicolasfella
      Subscribers: apol, nicolasfella, kdeconnect
      Tags: #kde_connect
      Differential Revision: https://phabricator.kde.org/D15978
  10. 05 Aug, 2018 1 commit
  11. 09 Jun, 2018 1 commit
    • Aleix Pol Gonzalez's avatar
      (WIP) Upgrade Telephony plugin to read SMS history (KDE side) · 31e93ef7
      Aleix Pol Gonzalez authored
      For real usecases of SMS support, we will almost always need access to the message history in some way
      Specifically resolve T8338
      Incidentally resolve T6651 since Telephony shall no longer create a notification
      Test Plan:
      - Build corresponding Android-side diff (D11698)
      - Build this diff
      Step 1: Does anything at all work?
      - Put a breakpoint in the handleBatchMessages method of the telephony plugin, ideally after constructing a Message object
      - Use DBus to poke /modules/kdeconnect/devices/<deviceID>/telephony.requestAllConversations()
      - Verify that the constructed Message is one you sent or received and that it is the most recent in the corresponding conversation
      Step 2: DBus
      - Open the Interface org.kde.kdeconnect.device.conversations of /modules/kdeconnect/devices/<deviceId>
      - Poke activeConversations and verify an empty array is returned
      - Poke requestAllConversationThreads
      - Poke activeConversations and verify that a list of numbers has been returned. These are conversationIds
      - Use a conversationId to call getFirstFromConversation
        - Verify that the returned Message object is one which you recognize
        - Note that if the message is an MMS it will be blank and meaningless. Try a different conversationId. MMS support "coming soon!"
      Step 3: SMS App
      - Use ccmake (or similar) to set SMSAPP_ENABLE to ON
      - Build the project
      - Run ./bin/kdeconnect-sms
      - Verify that the app shows a list of everyone you have an SMS conversation with (MMS messages are stripped out)
        - If you have the Contacts plugin working, verify that most contacts have their name and photo instead of their phone number
      Reviewers: #kde_connect, nicolasfella, apol
      Reviewed By: #kde_connect, nicolasfella, apol
      Subscribers: andyholmes, apol, nicolasfella, #kde_connect
      Tags: #kde_connect
      Maniphest Tasks: T8338, T6651
      Differential Revision: https://phabricator.kde.org/D11854