fix(server): remove isWatched from DTO (#8598)

* fix: remove isWatched

* chore: open api

mobile/openapi/doc/CreateLibraryDto.md

@@ -11,7 +11,6 @@ Name | Type | Description | Notes
 **exclusionPatterns** | **List<String>** |  | [optional] [default to const []]
 **importPaths** | **List<String>** |  | [optional] [default to const []]
 **isVisible** | **bool** |  | [optional] 
-**isWatched** | **bool** |  | [optional] 
 **name** | **String** |  | [optional] 
 **ownerId** | **String** |  | 
 **type** | [**LibraryType**](LibraryType.md) |  | 

mobile/openapi/lib/model/create_library_dto.dart

@@ -16,7 +16,6 @@ class CreateLibraryDto {
     this.exclusionPatterns = const [],
     this.importPaths = const [],
     this.isVisible,
-    this.isWatched,
     this.name,
     required this.ownerId,
     required this.type,
@@ -34,14 +33,6 @@ class CreateLibraryDto {
   ///
   bool? isVisible;
 
-  ///
-  /// Please note: This property should have been non-nullable! Since the specification file
-  /// does not include a default value (using the "default:" property), however, the generated
-  /// source code must fall back to having a nullable type.
-  /// Consider adding a "default:" property in the specification file to hide this note.
-  ///
-  bool? isWatched;
-
   ///
   /// Please note: This property should have been non-nullable! Since the specification file
   /// does not include a default value (using the "default:" property), however, the generated
@@ -59,7 +50,6 @@ class CreateLibraryDto {
     _deepEquality.equals(other.exclusionPatterns, exclusionPatterns) &&
     _deepEquality.equals(other.importPaths, importPaths) &&
     other.isVisible == isVisible &&
-    other.isWatched == isWatched &&
     other.name == name &&
     other.ownerId == ownerId &&
     other.type == type;
@@ -70,13 +60,12 @@ class CreateLibraryDto {
     (exclusionPatterns.hashCode) +
     (importPaths.hashCode) +
     (isVisible == null ? 0 : isVisible!.hashCode) +
-    (isWatched == null ? 0 : isWatched!.hashCode) +
     (name == null ? 0 : name!.hashCode) +
     (ownerId.hashCode) +
     (type.hashCode);
 
   @override
-  String toString() => 'CreateLibraryDto[exclusionPatterns=$exclusionPatterns, importPaths=$importPaths, isVisible=$isVisible, isWatched=$isWatched, name=$name, ownerId=$ownerId, type=$type]';
+  String toString() => 'CreateLibraryDto[exclusionPatterns=$exclusionPatterns, importPaths=$importPaths, isVisible=$isVisible, name=$name, ownerId=$ownerId, type=$type]';
 
   Map<String, dynamic> toJson() {
     final json = <String, dynamic>{};
@@ -87,11 +76,6 @@ class CreateLibraryDto {
     } else {
     //  json[r'isVisible'] = null;
     }
-    if (this.isWatched != null) {
-      json[r'isWatched'] = this.isWatched;
-    } else {
-    //  json[r'isWatched'] = null;
-    }
     if (this.name != null) {
       json[r'name'] = this.name;
     } else {
@@ -117,7 +101,6 @@ class CreateLibraryDto {
             ? (json[r'importPaths'] as Iterable).cast<String>().toList(growable: false)
             : const [],
         isVisible: mapValueOfType<bool>(json, r'isVisible'),
-        isWatched: mapValueOfType<bool>(json, r'isWatched'),
         name: mapValueOfType<String>(json, r'name'),
         ownerId: mapValueOfType<String>(json, r'ownerId')!,
         type: LibraryType.fromJson(json[r'type'])!,

mobile/openapi/test/create_library_dto_test.dart

@@ -31,11 +31,6 @@ void main() {
       // TODO
     });
 
-    // bool isWatched
-    test('to test the property `isWatched`', () async {
-      // TODO
-    });
-
     // String name
     test('to test the property `name`', () async {
       // TODO

open-api/immich-openapi-specs.json

@@ -8000,9 +8000,6 @@
           "isVisible": {
             "type": "boolean"
           },
-          "isWatched": {
-            "type": "boolean"
-          },
           "name": {
             "type": "string"
           },

open-api/typescript-sdk/src/fetch-client.ts

@@ -461,7 +461,6 @@ export type CreateLibraryDto = {
     exclusionPatterns?: string[];
     importPaths?: string[];
     isVisible?: boolean;
-    isWatched?: boolean;
     name?: string;
     ownerId: string;
     "type": LibraryType;

server/src/dtos/library.dto.ts

@@ -32,9 +32,6 @@ export class CreateLibraryDto {
   @ArrayUnique()
   @ArrayMaxSize(128)
   exclusionPatterns?: string[];
-
-  @ValidateBoolean({ optional: true })
-  isWatched?: boolean;
 }
 
 export class UpdateLibraryDto {

server/src/services/library.service.spec.ts

@@ -1058,14 +1058,6 @@ describe(LibraryService.name, () => {
 
         expect(libraryMock.create).not.toHaveBeenCalled();
       });
-
-      it('should not create watched', async () => {
-        await expect(
-          sut.create({ ownerId: authStub.admin.user.id, type: LibraryType.UPLOAD, isWatched: true }),
-        ).rejects.toBeInstanceOf(BadRequestException);
-
-        expect(storageMock.watch).not.toHaveBeenCalled();
-      });
     });
   });
 

server/src/services/library.service.ts

@@ -266,9 +266,6 @@ export class LibraryService extends EventEmitter {
         if (dto.exclusionPatterns && dto.exclusionPatterns.length > 0) {
           throw new BadRequestException('Upload libraries cannot have exclusion patterns');
         }
-        if (dto.isWatched) {
-          throw new BadRequestException('Upload libraries cannot be watched');
-        }
         break;
       }
     }

web/src/routes/admin/library-management/+page.svelte

@@ -118,7 +118,9 @@
 
   const handleCreate = async (ownerId: string) => {
     try {
-      const createdLibrary = await createLibrary({ createLibraryDto: { ownerId, type: LibraryType.External } });
+      const createdLibrary = await createLibrary({
+        createLibraryDto: { ownerId, type: LibraryType.External },
+      });
 
       notificationController.show({
         message: `Created library: ${createdLibrary.name}`,