diff --git a/mobile/lib/domain/services/remote_album.service.dart b/mobile/lib/domain/services/remote_album.service.dart index cb5f6489ea..dc3d608186 100644 --- a/mobile/lib/domain/services/remote_album.service.dart +++ b/mobile/lib/domain/services/remote_album.service.dart @@ -183,8 +183,10 @@ class RemoteAlbumService { return album.added.length; } - /// Adds an asset selection to an album, uploading any local-only assets - /// first and linking each one as soon as its upload completes. + /// !TODO The name here is not clear as we have addAssets method above, + /// which is only add remote assets to album, for the next PR, we will allow + /// adding local assets from album from the timeline as well with this flow. + /// So saving that for the next refactor Future addAssetsToAlbum({ required String albumId, required UserDto uploader, diff --git a/mobile/lib/presentation/pages/drift_remote_album.page.dart b/mobile/lib/presentation/pages/drift_remote_album.page.dart index 0046ce4097..89d3a8f037 100644 --- a/mobile/lib/presentation/pages/drift_remote_album.page.dart +++ b/mobile/lib/presentation/pages/drift_remote_album.page.dart @@ -39,7 +39,8 @@ class _RemoteAlbumPageState extends ConsumerState { } Future addAssets(BuildContext context) async { - final albumAssets = await ref.read(remoteAlbumProvider.notifier).getAssets(_album.id); + final notifier = ref.read(remoteAlbumProvider.notifier); + final albumAssets = await notifier.getAssets(_album.id); final newAssets = await context.pushRoute>( DriftAssetSelectionTimelineRoute(lockedSelectionAssets: albumAssets.toSet()), @@ -49,7 +50,7 @@ class _RemoteAlbumPageState extends ConsumerState { return; } - final added = await ref.read(remoteAlbumProvider.notifier).addAssetsToAlbum(_album.id, newAssets); + final added = await notifier.addAssetsToAlbum(_album.id, newAssets); if (added > 0 && context.mounted) { ImmichToast.show(