fix: add missing ApiProperty decorator to patchExternalSettings endpo…

…int (#1670)

src/samples/schemas/sample.schema.ts

@@ -16,16 +16,16 @@ export type SampleDocument = SampleClass & Document;
   timestamps: true,
 })
 export class SampleClass extends OwnableClass {
-  @ApiHideProperty()
-  @Prop({ type: String })
-  _id: string;
-
   /**
    * Globally unique identifier of a sample. This could be provided as an input value or generated by the system.
    */
   @Prop({ type: String, unique: true, required: true, default: () => uuidv4() })
   sampleId: string;
 
+  @ApiHideProperty()
+  @Prop({ type: String })
+  _id: string;
+
   /**
    * The owner of the sample.
    */

src/users/users.controller.ts

@@ -16,6 +16,7 @@ import {
   ApiBearerAuth,
   ApiBody,
   ApiOperation,
+  ApiParam,
   ApiResponse,
   ApiTags,
 } from "@nestjs/swagger";
@@ -304,11 +305,21 @@ export class UsersController {
       ability.can(Action.UserUpdateOwn, User) ||
       ability.can(Action.UserUpdateAny, User),
   )
+  @ApiParam({ name: "id", type: String, description: "User ID" })
+  @ApiBody({
+    schema: {
+      type: "object",
+      additionalProperties: true,
+      example: { field: "setting" },
+      description:
+        "External settings to update. This should be a key-value pair object containing the settings to modify.",
+    },
+  })
   @Patch("/:id/settings/external")
   async patchExternalSettings(
     @Req() request: Request,
     @Param("id") id: string,
-    @Body() externalSettings: Record<string, unknown>,
+    @Body() externalSettings = {},
   ): Promise<UserSettings | null> {
     await this.checkUserAuthorization(
       request,