Skip to content

Conversation

@gonfff
Copy link
Owner

@gonfff gonfff commented Jan 11, 2026

No description provided.

Copy link

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 pull request adds functionality to automatically refresh overdue next payment dates for subscriptions. When subscriptions are loaded or updated, the system now checks for any subscriptions with next payment dates in the past and recalculates them to the appropriate future date based on their billing cycle.

Changes:

  • Added new RefreshOverdueNextPaymentsUseCase to identify and update subscriptions with overdue next payment dates
  • Integrated the use case into SubscriptionsViewModel to automatically trigger on subscription updates
  • Added comprehensive test coverage for both the use case and view model integration

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
lib/application/subscriptions/refresh_overdue_next_payments_use_case.dart New use case that iterates through subscriptions and updates those with overdue next payment dates
lib/application/app_dependencies.dart Wired the new use case into the dependency injection system
lib/presentation/viewmodels/subscriptions_view_model.dart Integrated the use case to trigger automatically when subscriptions are updated via the stream
lib/presentation/screens/subscriptions_screen.dart Passed the new use case dependency to the view model
test/application/subscriptions/subscription_use_cases_test.dart Added unit test verifying the use case correctly updates overdue subscriptions
test/presentation/viewmodels/subscriptions_view_model_test.dart Added integration test verifying the use case is called when subscriptions update

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

@gonfff gonfff force-pushed the fix-next-payment-recount branch from b042f2f to 14c63dd Compare January 11, 2026 12:39
@gonfff gonfff merged commit c95093b into master Jan 11, 2026
1 check passed
@gonfff gonfff deleted the fix-next-payment-recount branch January 11, 2026 12:40
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