diff --git a/web/apps/photos/src/pages/gallery.tsx b/web/apps/photos/src/pages/gallery.tsx index 5e334edbf4..e9c1d8be93 100644 --- a/web/apps/photos/src/pages/gallery.tsx +++ b/web/apps/photos/src/pages/gallery.tsx @@ -667,11 +667,10 @@ const Page: React.FC = () => { if (notifyOtherFiles) { showMiniDialog({ - title: "Note", + title: pt("Note"), icon: , - message: pt( - "Files which are not owned by you were not removed", - ), + message: pt("Files added by other users were not removed"), + cancel: t("ok"), }); } })(); diff --git a/web/packages/new/photos/services/collection.ts b/web/packages/new/photos/services/collection.ts index a7b6267f8e..e49c0c0daf 100644 --- a/web/packages/new/photos/services/collection.ts +++ b/web/packages/new/photos/services/collection.ts @@ -737,12 +737,16 @@ export const removeOtherOtherNotSupportErrorMessage = * * 1. [Public] {@link removeFromCollection} - Handles both own and others * collections by delegating to the one of the following functions. + * * 2. [Public] {@link removeFromOwnCollection} - Handles both cases for own * collections by delegating to either "Move" or "Remove" + * * 3. [Private] {@link removeFromOthersCollection} - Handles both cases for * other's collections by delegating to "Remove", then if needed, also * throwing an error for the unsupported case. + * * 4. [Private] {@link removeOwnFilesFromOwnCollection} implements the "Move". + * * 5. [Private] {@link removeNonCollectionOwnerFiles} implements the "Remove". */ export const removeFromCollection = async ( @@ -780,7 +784,7 @@ const removeFromOthersCollection = async ( (f) => f.ownerID == userID, ); if (userFiles.length) { - await removeNonCollectionOwnerFiles(collectionID, nonUserFiles); + await removeNonCollectionOwnerFiles(collectionID, userFiles); } if (nonUserFiles.length) { throw new Error(removeOtherOtherNotSupportErrorMessage);