Skip to content

Real PullToTefresh (sync) state#144

Merged
rugged-bl merged 1 commit intomainfrom
real-pull-to-refresh-status
Jan 10, 2026
Merged

Real PullToTefresh (sync) state#144
rugged-bl merged 1 commit intomainfrom
real-pull-to-refresh-status

Conversation

@rugged-bl
Copy link
Contributor

No description provided.

@rugged-bl rugged-bl requested a review from Copilot January 10, 2026 16:05
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR implements real pull-to-refresh state tracking by introducing an EventsSyncStateHolder that monitors ongoing sync operations and reflects their status in the UI.

Changes:

  • Added EventsSyncStateHolder to track which events are currently syncing
  • Integrated sync state tracking into both iOS and Android EventsSyncManager implementations
  • Updated ExpensesViewModel to use the new PullToRefreshStateManager that combines sync state with user-triggered refresh

Reviewed changes

Copilot reviewed 21 out of 22 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
android/shared/feature/events/src/commonMain/kotlin/com/inwords/expenses/feature/events/domain/EventsSyncStateHolder.kt New class to hold and expose sync state for events
android/shared/feature/sync/src/iosMain/kotlin/com/inwords/expenses/feature/sync/data/EventsSyncManager.ios.kt Tracks syncing events in a MutableStateFlow and exposes sync state
android/shared/feature/sync/src/androidMain/kotlin/com/inwords/expenses/feature/sync/data/SyncManager.android.kt Implements sync state tracking via WorkManager tags
android/shared/feature/expenses/src/commonMain/kotlin/com/inwords/expenses/feature/expenses/ui/list/PullToRefreshStateManager.kt New manager combining sync state with user-triggered refresh delays
android/shared/feature/expenses/src/commonMain/kotlin/com/inwords/expenses/feature/expenses/ui/list/ExpensesViewModel.kt Refactored to use sync state holder and new refresh manager
android/shared/feature/sync/src/commonMain/kotlin/com/inwords/expenses/feature/sync/domain/EventsSyncObserver.kt Observes sync state and updates EventsSyncStateHolder
android/shared/feature/events/src/commonMain/kotlin/com/inwords/expenses/feature/events/api/EventsComponent.kt Renamed property from eventDeletionStateManager to eventDeletionStateManagerLazy and added eventsSyncStateHolderLazy
android/shared/integration/base/src/commonMain/kotlin/com/inwords/expenses/integration/base/MainNavHost.kt Updated references to use renamed lazy properties
android/shared/feature/sync/src/androidMain/kotlin/com/inwords/expenses/feature/sync/data/SyncWorkers.kt Added common tag for all sync workers and made constant internal
android/shared/feature/expenses/src/commonMain/kotlin/com/inwords/expenses/feature/expenses/domain/ExpensesInteractor.kt Renamed method from onRefreshExpensesAsync to enqueueAsyncExpensesRefresh

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@rugged-bl rugged-bl force-pushed the real-pull-to-refresh-status branch from 1e96881 to d916cef Compare January 10, 2026 16:10
@rugged-bl rugged-bl requested a review from Copilot January 10, 2026 16:13
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 21 out of 22 changed files in this pull request and generated 4 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@rugged-bl rugged-bl force-pushed the real-pull-to-refresh-status branch 2 times, most recently from d09a188 to 4e494a0 Compare January 10, 2026 16:40
@rugged-bl rugged-bl force-pushed the real-pull-to-refresh-status branch from 4e494a0 to 2bca5d5 Compare January 10, 2026 16:48
@rugged-bl rugged-bl merged commit dfde487 into main Jan 10, 2026
6 checks passed
@rugged-bl rugged-bl deleted the real-pull-to-refresh-status branch January 10, 2026 16:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants