refactor(placeholders): use slugify for default values

novuhq · Nov 19, 2024 · 1b8e0a7 · 1b8e0a7
1 parent f9b2199
commit 1b8e0a7
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/...ecases/validate-content/collect-placeholders/collect-placeholder-with-defaults.usecase.ts b/...ecases/validate-content/collect-placeholders/collect-placeholder-with-defaults.usecase.ts
@@ -1,4 +1,5 @@
 import { Injectable, InternalServerErrorException } from '@nestjs/common';
+import { slugify } from '@novu/shared';
 import { PlaceholderAggregation } from './placeholder.aggregation';
 import { HydrateEmailSchemaUseCase } from '../../../../environments-v1/usecases/output-renderers';
 import { CollectPlaceholderWithDefaultsCommand } from './collect-placeholder-with-defaults.command';
@@ -110,7 +111,7 @@ function extractLiquidJSPlaceholders(text: string) {
     } else {
       matches.push({
         placeholder: `{{${trimmedContent}}}`,
-        defaultValue,
+        defaultValue: slugify(defaultValue, { separator: '_' }),
       });
     }
   }