Merge pull request #81 from codex-team/fix/patch-note-settings

fix: fixed note settings patch

src/presentation/http/router/noteSettings.ts

@@ -5,16 +5,7 @@ import type NoteSettings from '@domain/entities/noteSettings.js';
 import notEmpty from '@infrastructure/utils/notEmpty.js';
 import useNoteResolver from '../middlewares/note/useNoteResolver.js';
 import type NoteService from '@domain/service/note.js';
-
-/**
- * Get note by id options
- */
-interface GetNoteSettingsByNodeIdOptions {
-  /**
-   * Note internal id
-   */
-  id: NoteInternalId;
-}
+import { NotePublicId } from '@domain/entities/note.js';
 
 /**
  * Interface for the note settings router.
@@ -55,9 +46,11 @@ const NoteSettingsRouter: FastifyPluginCallback<NoteSettingsRouterOptions> = (fa
    * Returns Note settings by note id. Note public id is passed in route params, and it converted to internal id via middleware
    */
   fastify.get<{
-    Params: GetNoteSettingsByNodeIdOptions,
+    Params: {
+      notePublicId: NotePublicId;
+    },
     Reply: NoteSettings
-  }>('/:id', {
+  }>('/:notePublicId', {
     preHandler: [
       noteIdResolver,
     ],
@@ -67,7 +60,6 @@ const NoteSettingsRouter: FastifyPluginCallback<NoteSettingsRouterOptions> = (fa
      */
     const noteId = request.noteId as number;
 
-
     const noteSettings = await noteSettingsService.getNoteSettingsByNoteId(noteId);
 
     /**
@@ -85,9 +77,11 @@ const NoteSettingsRouter: FastifyPluginCallback<NoteSettingsRouterOptions> = (fa
    */
   fastify.patch<{
     Body: Partial<NoteSettings>,
-    Params: GetNoteSettingsByNodeIdOptions,
+    Params: {
+      notePublicId: NotePublicId;
+    },
     Reply: NoteSettings,
-  }>('/:id', {
+  }>('/:notePublicId', {
     config: {
       policy: [
         'authRequired',