refactor: remove logout listener (#1357)

espresso-cash · Mar 30, 2024 · f374001 · f374001
1 parent b08f877
commit f374001
Show file tree

Hide file tree

Showing 5 changed files with 14 additions and 51 deletions.
diff --git a/packages/espressocash_app/lib/features/accounts/module.dart b/packages/espressocash_app/lib/features/accounts/module.dart
diff --git a/packages/espressocash_app/lib/features/incoming_link_payments/data/ilp_repository.dart b/packages/espressocash_app/lib/features/incoming_link_payments/data/ilp_repository.dart
@@ -4,19 +4,21 @@ import 'package:dfunc/dfunc.dart';
 import 'package:drift/drift.dart';
 import 'package:espressocash_common/espressocash_common.dart';
 import 'package:fast_immutable_collections/fast_immutable_collections.dart';
+import 'package:get_it/get_it.dart';
 import 'package:injectable/injectable.dart';
 import 'package:sentry_flutter/sentry_flutter.dart';
 import 'package:solana/base58.dart';
 import 'package:solana/encoder.dart';
 
 import '../../../data/db/db.dart';
 import '../../../data/db/mixins.dart';
+import '../../authenticated/auth_scope.dart';
 import '../../escrow/models/escrow_private_key.dart';
 import '../../transactions/models/tx_results.dart';
 import '../models/incoming_link_payment.dart';
 
-@injectable
-class ILPRepository {
+@Singleton(scope: authScope)
+class ILPRepository implements Disposable {
   const ILPRepository(this._db);
 
   final MyDatabase _db;
@@ -50,7 +52,8 @@ class ILPRepository {
     await _db.into(_db.iLPRows).insertOnConflictUpdate(await payment.toDto());
   }
 
-  Future<void> clear() => _db.delete(_db.iLPRows).go();
+  @override
+  Future<void> onDispose() => _db.delete(_db.iLPRows).go();
 
   Stream<IList<IncomingLinkPayment>> watchTxCreated() => _watchWithStatuses([
         ILPStatusDto.txCreated,

diff --git a/packages/espressocash_app/lib/features/incoming_link_payments/module.dart b/packages/espressocash_app/lib/features/incoming_link_payments/module.dart
@@ -3,9 +3,7 @@ import 'package:nested/nested.dart';
 import 'package:provider/provider.dart';
 
 import '../../di.dart';
-import '../accounts/module.dart';
 import '../balances/widgets/context_ext.dart';
-import 'data/ilp_repository.dart';
 import 'services/tx_created_watcher.dart';
 import 'services/tx_sent_watcher.dart';
 
@@ -28,9 +26,6 @@ class ILPModule extends SingleChildStatelessWidget {
             dispose: (_, value) => value.dispose(),
           ),
         ],
-        child: LogoutListener(
-          onLogout: (_) => sl<ILPRepository>().clear(),
-          child: child ?? const SizedBox.shrink(),
-        ),
+        child: child,
       );
 }
diff --git a/packages/espressocash_app/lib/features/outgoing_link_payments/data/repository.dart b/packages/espressocash_app/lib/features/outgoing_link_payments/data/repository.dart
@@ -4,20 +4,22 @@ import 'package:dfunc/dfunc.dart';
 import 'package:drift/drift.dart';
 import 'package:espressocash_common/espressocash_common.dart';
 import 'package:fast_immutable_collections/fast_immutable_collections.dart';
+import 'package:get_it/get_it.dart';
 import 'package:injectable/injectable.dart';
 import 'package:sentry_flutter/sentry_flutter.dart';
 import 'package:solana/base58.dart';
 import 'package:solana/encoder.dart';
 
 import '../../../data/db/db.dart';
 import '../../../data/db/mixins.dart';
+import '../../authenticated/auth_scope.dart';
 import '../../escrow/models/escrow_private_key.dart';
 import '../../tokens/token_list.dart';
 import '../../transactions/models/tx_results.dart';
 import '../models/outgoing_link_payment.dart';
 
-@injectable
-class OLPRepository {
+@Singleton(scope: authScope)
+class OLPRepository implements Disposable {
   const OLPRepository(this._db, this._tokens);
 
   final MyDatabase _db;
@@ -91,7 +93,8 @@ class OLPRepository {
         OLPStatusDto.cancelTxSent,
       ]);
 
-  Future<void> clear() => _db.delete(_db.oLPRows).go();
+  @override
+  Future<void> onDispose() => _db.delete(_db.oLPRows).go();
 
   Stream<IList<OutgoingLinkPayment>> _watchWithStatuses(
     Iterable<OLPStatusDto> statuses,

diff --git a/packages/espressocash_app/lib/features/outgoing_link_payments/module.dart b/packages/espressocash_app/lib/features/outgoing_link_payments/module.dart
@@ -4,9 +4,7 @@ import 'package:provider/provider.dart';
 
 import '../../di.dart';
 import '../accounts/models/account.dart';
-import '../accounts/module.dart';
 import '../balances/widgets/context_ext.dart';
-import 'data/repository.dart';
 import 'services/cancel_tx_created_watcher.dart';
 import 'services/cancel_tx_sent_watcher.dart';
 import 'services/tx_confirmed_watcher.dart';
@@ -58,9 +56,6 @@ class OLPModule extends SingleChildStatelessWidget {
             dispose: (_, value) => value.dispose(),
           ),
         ],
-        child: LogoutListener(
-          onLogout: (_) => sl<OLPRepository>().clear(),
-          child: child,
-        ),
+        child: child,
       );
 }