Skip to content

Transaction lifecycle updates - #211

Merged
austin047 merged 5 commits into
devfrom
feat/transaction-lifecycle
Jul 15, 2026
Merged

Transaction lifecycle updates#211
austin047 merged 5 commits into
devfrom
feat/transaction-lifecycle

Conversation

@austin047

@austin047 austin047 commented Jul 15, 2026

Copy link
Copy Markdown
Collaborator

Description

  • Added the refund to the transaction
  • Added mark as recurring to the transaction
  • Added reminder
  • Updated to the AI chat landing page
WhatsApp Image 2026-07-15 at 10 38 35 (1) WhatsApp Image 2026-07-15 at 10 38 35 WhatsApp Image 2026-07-15 at 10 38 34 (1) WhatsApp Image 2026-07-15 at 10 38 34 WhatsApp Image 2026-07-15 at 10 38 51

Type of Change

  • ✨ New feature (non-breaking change which adds functionality)
  • 🛠️ Bug fix (non-breaking change which fixes an issue)
  • ❌ Breaking change (fix or feature that would cause existing functionality to change)
  • 🧹 Code refactor
  • ✅ Build configuration change
  • 📝 Documentation
  • 🗑️ Chore

@austin047
austin047 marked this pull request as ready for review July 15, 2026 09:40
@austin047
austin047 requested a review from nfebe July 15, 2026 09:40
@sourceant

sourceant Bot commented Jul 15, 2026

Copy link
Copy Markdown

Code Review Summary

This PR introduces a comprehensive lifecycle update for transactions, including refund support, recurring transaction rules, and a new 'Reminders' feature. It involves a database schema bump to version 7 and substantial UI additions.

🚀 Key Improvements

  • Support for linked refunds (linking income to original expenses).
  • Implementation of recurring transaction rules (period, interval, end dates).
  • Full CRUD and sync support for a new Reminders entity.
  • Updated AI chat landing page with enhanced suggestion chips and icons.

💡 Minor Suggestions

  • Add safeguard limits to infinite loops in remote data sources.
  • Avoid using .then() inside unawaited blocks in repositories to prevent complex race conditions.

@sourceant sourceant Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Review complete. See the overview comment for a summary.

Comment thread lib/data/datasources/reminder/reminder_remote_datasource.dart
Comment thread lib/data/repositories/transaction_repository_impl.dart
Comment thread lib/presentation/reminders/add_reminder_screen.dart

@nfebe nfebe left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

We need to review the compactness of the intent and the add page in general. I leftt a comment regarding this in the group chat.

@austin047
austin047 merged commit 7ad4de9 into dev Jul 15, 2026
3 checks passed
@austin047
austin047 deleted the feat/transaction-lifecycle branch July 15, 2026 15:32
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