From a2c82e63f83198f943a3aa66c3b235bcf9cd135f Mon Sep 17 00:00:00 2001
From: Bruno Mello <bruno.barros.mello@gmail.com>
Date: Fri, 19 Feb 2021 13:57:47 -0300
Subject: [PATCH 1/3] fix: remove article from wrapping content

---
 __tests__/mocks/html.ts                       | 28 ++++++++-----------
 .../html-wrapper/options-html-wrapper.ts      |  8 +-----
 2 files changed, 13 insertions(+), 23 deletions(-)
diff --git a/__tests__/mocks/html.ts b/__tests__/mocks/html.ts
index f2a5c01..5f257c2 100644
--- a/__tests__/mocks/html.ts
+++ b/__tests__/mocks/html.ts
@@ -265,17 +265,13 @@ const HEADER = `\
 </header>
 `;
 
-const ARTICLE_CONTENT = `\
-<article class="page sans">
-  ${HEADER}
-  <p>Hello World</p>
-</article>
+const CONTENT_WITH_HEADER = `\
+${HEADER}
+<p>Hello World</p>
 `;
 
-const ARTICLE_CONTENT_WITHOUT_HEADER = `\
-<article class="page sans">
-  <p>Hello World</p>
-</article>
+const CONTENT_WITHOUT_HEADER = `\
+<p>Hello World</p>
 `;
 
 export const FULL_DOCUMENT = `
@@ -289,7 +285,7 @@ export const FULL_DOCUMENT = `
     <link href="https://unpkg.com/prismjs@1.22.0/themes/prism.css" rel="stylesheet">
   </head>
   <body>
-    ${ARTICLE_CONTENT}
+    ${CONTENT_WITH_HEADER}
     <script src="https://unpkg.com/prismjs@1.22.0/components/prism-core.min.js"></script>
     <script src="https://unpkg.com/prismjs@1.22.0/plugins/autoloader/prism-autoloader.min.js"></script>
     <script>
@@ -316,7 +312,7 @@ export const DOCUMENT_WITHOUT_TITLE = `
     <link href="https://unpkg.com/prismjs@1.22.0/themes/prism.css" rel="stylesheet">
   </head>
   <body>
-    ${ARTICLE_CONTENT}
+    ${CONTENT_WITH_HEADER}
     <script src="https://unpkg.com/prismjs@1.22.0/components/prism-core.min.js"></script>
     <script src="https://unpkg.com/prismjs@1.22.0/plugins/autoloader/prism-autoloader.min.js"></script>
     <script>
@@ -343,7 +339,7 @@ export const DOCUMENT_WITHOUT_CSS = `
     <link href="https://unpkg.com/prismjs@1.22.0/themes/prism.css" rel="stylesheet">
   </head>
   <body>
-    ${ARTICLE_CONTENT}
+    ${CONTENT_WITH_HEADER}
     <script src="https://unpkg.com/prismjs@1.22.0/components/prism-core.min.js"></script>
     <script src="https://unpkg.com/prismjs@1.22.0/plugins/autoloader/prism-autoloader.min.js"></script>
     <script>
@@ -369,7 +365,7 @@ export const DOCUMENT_METADATA = `
     <link href="https://unpkg.com/prismjs@1.22.0/themes/prism.css" rel="stylesheet">
   </head>
   <body>
-    ${ARTICLE_CONTENT}
+    ${CONTENT_WITH_HEADER}
     <script src="https://unpkg.com/prismjs@1.22.0/components/prism-core.min.js"></script>
     <script src="https://unpkg.com/prismjs@1.22.0/plugins/autoloader/prism-autoloader.min.js"></script>
     <script>
@@ -396,7 +392,7 @@ export const DOCUMENT_WITHOUT_SCRIPTS = `
     <title>Simple Page Test</title>
   </head>
   <body>
-    ${ARTICLE_CONTENT}
+    ${CONTENT_WITH_HEADER}
   </body>
 </html>
 `;
@@ -412,7 +408,7 @@ export const FULL_DOCUMENT_WITHOUT_HEADER_IN_BODY = `
     <link href="https://unpkg.com/prismjs@1.22.0/themes/prism.css" rel="stylesheet">
   </head>
   <body>
-    ${ARTICLE_CONTENT_WITHOUT_HEADER}
+    ${CONTENT_WITHOUT_HEADER}
     <script src="https://unpkg.com/prismjs@1.22.0/components/prism-core.min.js"></script>
     <script src="https://unpkg.com/prismjs@1.22.0/plugins/autoloader/prism-autoloader.min.js"></script>
     <script>
@@ -429,7 +425,7 @@ export const FULL_DOCUMENT_WITHOUT_HEADER_IN_BODY = `
 </html>
 `;
 
-export const BODY_ONLY = ARTICLE_CONTENT_WITHOUT_HEADER;
+export const BODY_ONLY = CONTENT_WITHOUT_HEADER;
 
 export const HEADER_WITH_TITLE_ONLY = `\
 <header>
diff --git a/src/data/usecases/html-wrapper/options-html-wrapper.ts b/src/data/usecases/html-wrapper/options-html-wrapper.ts
index 4a3a459..9154ea2 100644
--- a/src/data/usecases/html-wrapper/options-html-wrapper.ts
+++ b/src/data/usecases/html-wrapper/options-html-wrapper.ts
@@ -13,11 +13,7 @@ export class OptionsHtmlWrapper implements HtmlWrapper {
   }
 
   wrapHtml(pageProps: PageProps, html: string): string {
-    if (this._options.bodyContentOnly)
-      return `\
-<article class="page sans">
-${html}
-</article>`;
+    if (this._options.bodyContentOnly) return `${html}`;
 
     const title = pageProps.title;
 
@@ -26,10 +22,8 @@ ${html}
 <html>
 ${this._headFromTemplate(title)}
 <body>
-<article class="page sans">
 ${!this._options.excludeHeaderFromBody ? new HeaderFromTemplate(pageProps).toHeader() : ''}
 ${html}
-</article>
 ${!this._options.excludeScripts ? SCRIPTS : ''}
 </body>
 </html>`;

From 8d1d2fea714efd4451d74756c10db375a65894c5 Mon Sep 17 00:00:00 2001
From: Alexandre Nunes <alexandrenunes@poli.ufrj.br>
Date: Mon, 1 Mar 2021 02:00:58 -0300
Subject: [PATCH 2/3] refactor: remove unnecessary template string

---
 package-lock.json                                      | 6 ++----
 package.json                                           | 3 +--
 src/data/usecases/html-wrapper/options-html-wrapper.ts | 2 +-
 3 files changed, 4 insertions(+), 7 deletions(-)

diff --git a/package-lock.json b/package-lock.json
index f5dc2af..2cb9d95 100644
--- a/package-lock.json
+++ b/package-lock.json
@@ -1,12 +1,12 @@
 {
   "name": "notion-page-to-html",
-  "version": "1.0.2",
+  "version": "1.0.3",
   "lockfileVersion": 2,
   "requires": true,
   "packages": {
     "": {
       "name": "notion-page-to-html",
-      "version": "1.0.2",
+      "version": "1.0.3",
       "license": "MIT",
       "devDependencies": {
         "@types/jest": "^26.0.15",
@@ -674,7 +674,6 @@
         "jest-resolve": "^26.6.1",
         "jest-util": "^26.6.1",
         "jest-worker": "^26.6.1",
-        "node-notifier": "^8.0.0",
         "slash": "^3.0.0",
         "source-map": "^0.6.0",
         "string-length": "^4.0.1",
@@ -4109,7 +4108,6 @@
         "@types/node": "*",
         "anymatch": "^3.0.3",
         "fb-watchman": "^2.0.0",
-        "fsevents": "^2.1.2",
         "graceful-fs": "^4.2.4",
         "jest-regex-util": "^26.0.0",
         "jest-serializer": "^26.5.0",
diff --git a/package.json b/package.json
index 4e921ce..c2bbebe 100644
--- a/package.json
+++ b/package.json
@@ -44,6 +44,5 @@
     "prettier": "^2.1.2",
     "ts-jest": "^26.4.3",
     "typescript": "^4.0.5"
-  },
-  "dependencies": {}
+  }
 }
diff --git a/src/data/usecases/html-wrapper/options-html-wrapper.ts b/src/data/usecases/html-wrapper/options-html-wrapper.ts
index 9154ea2..7e55988 100644
--- a/src/data/usecases/html-wrapper/options-html-wrapper.ts
+++ b/src/data/usecases/html-wrapper/options-html-wrapper.ts
@@ -13,7 +13,7 @@ export class OptionsHtmlWrapper implements HtmlWrapper {
   }
 
   wrapHtml(pageProps: PageProps, html: string): string {
-    if (this._options.bodyContentOnly) return `${html}`;
+    if (this._options.bodyContentOnly) return html;
 
     const title = pageProps.title;
 

From 8630467871e1506f28685b71056839529ffcaa58 Mon Sep 17 00:00:00 2001
From: Alexandre Nunes <alexandrenunes@poli.ufrj.br>
Date: Mon, 1 Mar 2021 02:11:50 -0300
Subject: [PATCH 3/3] feat: new package version. It's now 1.1.0

---
 package.json | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/package.json b/package.json
index c2bbebe..dfe4054 100644
--- a/package.json
+++ b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "notion-page-to-html",
-  "version": "1.0.3",
+  "version": "1.1.0",
   "description": "It converts public notion pages to html from url",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",
