Skip to content

Commit 4cd691d

Browse files
committed
refactor(data_migration): remove engagement migration
- Remove engagement migration logic from DemoDataMigrationService - Remove engagementRepository dependency from DemoDataMigrationService - Update bootstrap.dart to reflect changes in DemoDataMigrationService
1 parent ae0563c commit 4cd691d

File tree

2 files changed

+0
-40
lines changed

2 files changed

+0
-40
lines changed

lib/app/services/demo_data_migration_service.dart

Lines changed: 0 additions & 39 deletions
Original file line numberDiff line numberDiff line change
@@ -16,16 +16,13 @@ class DemoDataMigrationService {
1616
required DataRepository<AppSettings> appSettingsRepository,
1717
required DataRepository<UserContentPreferences>
1818
userContentPreferencesRepository,
19-
required DataRepository<Engagement> engagementRepository,
2019
}) : _appSettingsRepository = appSettingsRepository,
2120
_userContentPreferencesRepository = userContentPreferencesRepository,
22-
_engagementRepository = engagementRepository,
2321
_logger = Logger('DemoDataMigrationService');
2422

2523
final DataRepository<AppSettings> _appSettingsRepository;
2624
final DataRepository<UserContentPreferences>
2725
_userContentPreferencesRepository;
28-
final DataRepository<Engagement> _engagementRepository;
2926
final Logger _logger;
3027

3128
/// Migrates user settings and content preferences from an old anonymous
@@ -148,41 +145,5 @@ class DemoDataMigrationService {
148145
s,
149146
);
150147
}
151-
152-
// Migrate Engagements
153-
try {
154-
final oldEngagements = await _engagementRepository.readAll(
155-
userId: oldUserId,
156-
);
157-
for (final oldEngagement in oldEngagements.items) {
158-
final newEngagement = oldEngagement.copyWith(userId: newUserId);
159-
try {
160-
await _engagementRepository.create(
161-
item: newEngagement,
162-
userId: newUserId,
163-
);
164-
} on ConflictException {
165-
/* ignore, already exists */
166-
}
167-
}
168-
// Delete all old engagements at once after migration.
169-
await Future.wait(
170-
oldEngagements.items.map(
171-
(e) => _engagementRepository.delete(id: e.id, userId: oldUserId),
172-
),
173-
);
174-
_logger.fine(
175-
'[DemoDataMigrationService] ${oldEngagements.items.length} '
176-
'engagements migrated successfully from $oldUserId to $newUserId.',
177-
);
178-
} catch (e, s) {
179-
_logger.severe(
180-
'[DemoDataMigrationService] Error migrating engagements from '
181-
'$oldUserId to $newUserId: $e',
182-
e,
183-
s,
184-
);
185-
}
186-
187148
}
188149
}

lib/bootstrap.dart

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -518,7 +518,6 @@ Future<Widget> bootstrap(
518518
? DemoDataMigrationService(
519519
appSettingsRepository: appSettingsRepository,
520520
userContentPreferencesRepository: userContentPreferencesRepository,
521-
engagementRepository: engagementRepository,
522521
)
523522
: null;
524523
logger.fine(

0 commit comments

Comments
 (0)