Merge pull request #632 from Automattic/issue/303-sync-skip-time-sett…

…ings Allow syncing of skip time settings
Automattic · Dec 9, 2022 · 10eaa02 · 10eaa02
2 parents a0a30d5 + 07b004e
commit 10eaa02
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 2 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -4,6 +4,8 @@
 * Bug Fixes:
     *   Fixed podcast date format
         ([#477](https://github.com/Automattic/pocket-casts-android/pull/477)).
+    *   Fix unable to permanently change "Skip back time" setting
+        ([#632](https://github.com/Automattic/pocket-casts-android/pull/632)).
 
 7.28
 -----

diff --git a/...ettings/src/main/java/au/com/shiftyjelly/pocketcasts/settings/PlaybackSettingsFragment.kt b/...ettings/src/main/java/au/com/shiftyjelly/pocketcasts/settings/PlaybackSettingsFragment.kt
@@ -155,7 +155,10 @@ class PlaybackSettingsFragment : BaseFragment() {
                         saved = settings.skipForwardInSecsObservable
                             .subscribeAsState(settings.getSkipForwardInSecs())
                             .value,
-                        onSave = settings::setSkipForwardInSec
+                        onSave = {
+                            settings.setSkipForwardNeedsSync(true)
+                            settings.setSkipForwardInSec(it)
+                        }
                     )
 
                     // Skip back time
@@ -164,7 +167,10 @@ class PlaybackSettingsFragment : BaseFragment() {
                         saved = settings.skipBackwardInSecsObservable
                             .subscribeAsState(settings.getSkipBackwardInSecs())
                             .value,
-                        onSave = settings::setSkipBackwardInSec
+                        onSave = {
+                            settings.setSkipBackNeedsSync(true)
+                            settings.setSkipBackwardInSec(it)
+                        }
                     )
 
                     KeepScreenAwake(