[PR #467] [MERGED] Fix multiple sync of a message #485

Closed
opened 2026-02-28 01:24:47 +03:00 by kerem · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/ushahidi/SMSSync/pull/467
Author: @eyedol
Created: 2/13/2017
Status: Merged
Merged: 2/14/2017
Merged by: @eyedol

Base: masterHead: 461-fix-multiple-sync-of-a-message


📝 Commits (1)

  • 9d00afa Disable manual sync operation when background sync is active

📊 Changes

35 files changed (+503 additions, -425 deletions)

View changed files

📝 .travis.yml (+4 -3)
📝 dependencies/dependencies.gradle (+17 -14)
📝 smssync/build.gradle (+6 -5)
📝 smssync/src/main/java/org/addhen/smssync/presentation/App.java (+0 -3)
smssync/src/main/java/org/addhen/smssync/presentation/AppBus.java (+0 -63)
📝 smssync/src/main/java/org/addhen/smssync/presentation/receiver/SmsDeliveredReceiver.java (+2 -1)
📝 smssync/src/main/java/org/addhen/smssync/presentation/receiver/SmsSentReceiver.java (+2 -2)
📝 smssync/src/main/java/org/addhen/smssync/presentation/service/BaseWakefulIntentService.java (+0 -2)
📝 smssync/src/main/java/org/addhen/smssync/presentation/service/ServiceConstants.java (+6 -4)
📝 smssync/src/main/java/org/addhen/smssync/presentation/service/SmsReceiverService.java (+8 -7)
📝 smssync/src/main/java/org/addhen/smssync/presentation/service/SyncPendingMessagesService.java (+43 -10)
📝 smssync/src/main/java/org/addhen/smssync/presentation/util/Utility.java (+78 -19)
📝 smssync/src/main/java/org/addhen/smssync/presentation/view/ui/activity/GettingStartedActivity.java (+7 -7)
📝 smssync/src/main/java/org/addhen/smssync/presentation/view/ui/activity/MainActivity.java (+5 -4)
📝 smssync/src/main/java/org/addhen/smssync/presentation/view/ui/activity/SettingsActivity.java (+2 -2)
📝 smssync/src/main/java/org/addhen/smssync/presentation/view/ui/adapter/LogAdapter.java (+2 -2)
📝 smssync/src/main/java/org/addhen/smssync/presentation/view/ui/adapter/MessageAdapter.java (+12 -13)
📝 smssync/src/main/java/org/addhen/smssync/presentation/view/ui/adapter/WebServiceAdapter.java (+4 -4)
📝 smssync/src/main/java/org/addhen/smssync/presentation/view/ui/fragment/AddKeywordFragment.java (+17 -17)
📝 smssync/src/main/java/org/addhen/smssync/presentation/view/ui/fragment/AddPhoneNumberFilterFragment.java (+18 -8)

...and 15 more files

📄 Description

Fixes #461

This PR makes the following changes:

  • Use broadcast receiver to update UI with respect to sync status
  • Upgrade Raiburari
  • Use progress bar to notify users of failed message with status count
  • Refactor how notification is built to make it more resuable
  • Fix builds on travis and wercker

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/ushahidi/SMSSync/pull/467 **Author:** [@eyedol](https://github.com/eyedol) **Created:** 2/13/2017 **Status:** ✅ Merged **Merged:** 2/14/2017 **Merged by:** [@eyedol](https://github.com/eyedol) **Base:** `master` ← **Head:** `461-fix-multiple-sync-of-a-message` --- ### 📝 Commits (1) - [`9d00afa`](https://github.com/ushahidi/SMSSync/commit/9d00afa1e476d15ddf6412c07f4b990e7c024006) Disable manual sync operation when background sync is active ### 📊 Changes **35 files changed** (+503 additions, -425 deletions) <details> <summary>View changed files</summary> 📝 `.travis.yml` (+4 -3) 📝 `dependencies/dependencies.gradle` (+17 -14) 📝 `smssync/build.gradle` (+6 -5) 📝 `smssync/src/main/java/org/addhen/smssync/presentation/App.java` (+0 -3) ➖ `smssync/src/main/java/org/addhen/smssync/presentation/AppBus.java` (+0 -63) 📝 `smssync/src/main/java/org/addhen/smssync/presentation/receiver/SmsDeliveredReceiver.java` (+2 -1) 📝 `smssync/src/main/java/org/addhen/smssync/presentation/receiver/SmsSentReceiver.java` (+2 -2) 📝 `smssync/src/main/java/org/addhen/smssync/presentation/service/BaseWakefulIntentService.java` (+0 -2) 📝 `smssync/src/main/java/org/addhen/smssync/presentation/service/ServiceConstants.java` (+6 -4) 📝 `smssync/src/main/java/org/addhen/smssync/presentation/service/SmsReceiverService.java` (+8 -7) 📝 `smssync/src/main/java/org/addhen/smssync/presentation/service/SyncPendingMessagesService.java` (+43 -10) 📝 `smssync/src/main/java/org/addhen/smssync/presentation/util/Utility.java` (+78 -19) 📝 `smssync/src/main/java/org/addhen/smssync/presentation/view/ui/activity/GettingStartedActivity.java` (+7 -7) 📝 `smssync/src/main/java/org/addhen/smssync/presentation/view/ui/activity/MainActivity.java` (+5 -4) 📝 `smssync/src/main/java/org/addhen/smssync/presentation/view/ui/activity/SettingsActivity.java` (+2 -2) 📝 `smssync/src/main/java/org/addhen/smssync/presentation/view/ui/adapter/LogAdapter.java` (+2 -2) 📝 `smssync/src/main/java/org/addhen/smssync/presentation/view/ui/adapter/MessageAdapter.java` (+12 -13) 📝 `smssync/src/main/java/org/addhen/smssync/presentation/view/ui/adapter/WebServiceAdapter.java` (+4 -4) 📝 `smssync/src/main/java/org/addhen/smssync/presentation/view/ui/fragment/AddKeywordFragment.java` (+17 -17) 📝 `smssync/src/main/java/org/addhen/smssync/presentation/view/ui/fragment/AddPhoneNumberFilterFragment.java` (+18 -8) _...and 15 more files_ </details> ### 📄 Description Fixes #461 This `PR` makes the following changes: - Use broadcast receiver to update UI with respect to sync status - Upgrade Raiburari - Use progress bar to notify users of failed message with status count - Refactor how notification is built to make it more resuable - Fix builds on travis and wercker --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-02-28 01:24:47 +03:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/SMSSync#485
No description provided.