From 335021b9bb967b689b179f522ab47c0e2279a383 Mon Sep 17 00:00:00 2001 From: Supan Adit Pratama Date: Sat, 2 Nov 2024 00:58:23 +0700 Subject: [PATCH] chore: add other notes in custom crud controller --- libs/skeleton/src/skeleton-crud.controller.ts | 1 + 1 file changed, 1 insertion(+) diff --git a/libs/skeleton/src/skeleton-crud.controller.ts b/libs/skeleton/src/skeleton-crud.controller.ts index c3c4995..92a579d 100644 --- a/libs/skeleton/src/skeleton-crud.controller.ts +++ b/libs/skeleton/src/skeleton-crud.controller.ts @@ -81,6 +81,7 @@ export class SkeletonCRUDController implements ISkeletonCRUDController { // - I know this is not recommended but.... is there any way to pass custom unique identifier ? // - Everyone still can use SkeletonCRUDController with no issue if don't want or don't like this approach, but... unique identifier must ID for sure and the type should either UUID, String or Number // - Correct me if I wrong. I already read the main repository of NestJS and they use Reflect for passing some metadata ( I know how to do it ) but... it still not possible for dynamic unique identifier +// - Using environment variable is possible to set in the decorator and been tested but... it could lead to human error and more issue than using this which already automated and configured // Known issue: // - OpenAPI ( Swagger ) cannot read any decorator inside this export const CustomCRUDController = (options?: ControllerOption): any => {