new strategy

FranciscoMoretti · Oct 9, 2024 · ff5272e · ff5272e
1 parent 443c32c
commit ff5272e
Show file tree

Hide file tree

Showing 4 changed files with 18 additions and 0 deletions.
diff --git a/.changeset/modern-boats-talk.md b/.changeset/modern-boats-talk.md
@@ -0,0 +1,5 @@
+---
+"notion-downloader": patch
+---
+
+Created a urlEncoding naming strategy
diff --git a/packages/notion-downloader/src/config/schema.ts b/packages/notion-downloader/src/config/schema.ts
@@ -26,6 +26,7 @@ export const MarkdownNamingStrategyNames = z.enum([
   "githubSlug",
   "notionSlug",
   "title",
+  "urlEncoding",
 ])
 export type MarkdownNamingStrategyNames = z.infer<
   typeof MarkdownNamingStrategyNames

diff --git a/packages/notion-downloader/src/namingStrategy/getNamingStrategy.ts b/packages/notion-downloader/src/namingStrategy/getNamingStrategy.ts
@@ -14,6 +14,7 @@ import {
   GuidNamingStrategy,
   NotionSlugNamingStrategy,
   TitleNamingStrategy,
+  UrlEncodingNamingStrategy,
 } from "./namingStrategies"
 
 export function getAssetNamingStrategy(
@@ -42,6 +43,8 @@ export function getMarkdownNamingStrategy(
       return new TitleNamingStrategy()
     case AllNamingSchemaName.enum.guid:
       return new GuidNamingStrategy()
+    case MarkdownNamingStrategyNames.enum.urlEncoding:
+      return new UrlEncodingNamingStrategy()
     case MarkdownNamingStrategyNames.enum.githubSlug:
       return new GithubSlugNamingStrategy(slugProperty)
     case MarkdownNamingStrategyNames.enum.notionSlug:

diff --git a/packages/notion-downloader/src/namingStrategy/namingStrategies.ts b/packages/notion-downloader/src/namingStrategy/namingStrategies.ts
@@ -97,3 +97,12 @@ export class TitleNamingStrategy extends NamingStrategy {
       .replaceAll("--", "-")
   }
 }
+
+export class UrlEncodingNamingStrategy extends NamingStrategy {
+  constructor() {
+    super([ObjectType.enum.page, ObjectType.enum.database])
+  }
+  protected _nameForObject(notionObject: NotionDatabase | NotionPage): string {
+    return encodeURIComponent(notionObject.title)
+  }
+}