Fixed: front matter route can use / to define the homepage (#721

)

* Define root index of the project via `route` parameter

Fix #673

* Add tests

* Add changelog notes for #721

[skip ci]

CHANGELOG.md

@@ -16,6 +16,10 @@
   ([ref #433](https://github.com/MoOx/phenomic/issues/433) - @MoOx)
 - Changed: error messages for bad configuration are now more readable
   ([#672](https://github.com/MoOx/phenomic/issues/672) - @MoOx)
+- Fixed: front matter ``route`` can use ``/`` to define the homepage
+  ([#721](https://github.com/MoOx/phenomic/pull/721) - @thangngoc89)
+  single string is passed as a child.
+  (@MoOx)
 - Added: 🚀 Hard source Webpack plugin to improve performance by more than 300%
   You can enable this **experimental** feature by add
   ``"webpackHardCache": true`` in your ``package.json`` ``phenomic`` section.

src/components/PageContainer/component.js

@@ -27,7 +27,10 @@ type Context = {
 
 // react-router does not return leading and trailing slashes
 // so we need to normalize according to collection data
-const splatToUrl = (string) => ("/" + urlify(string))
+const splatToUrl = (string) => {
+  const url = "/" + urlify(string)
+  return (url === "//") ? "/" : url
+}
 
 const isDevelopment = (): boolean => process.env.NODE_ENV !== "production"
 const isClient = (): boolean => typeof window !== "undefined"

src/phenomic-loader/__tests__/fixtures/custom-route-root-index.md

@@ -0,0 +1,4 @@
+---
+title: custom route root index
+route: /
+---

src/phenomic-loader/__tests__/index.js

@@ -86,6 +86,17 @@ test.cb("phenomic loader", (t) => {
         "should create a proper json for custom route with an extension"
       )
 
+      const customRouteRootIndex = stats.compilation.assets[
+        //    fixtures/custom-route-root-index.md
+        // -> fixtures/index.html
+        "index.html" +
+        ".8817d2a1fab9dfb9b4b52cd6ee7529ab.json"
+      ]
+      t.truthy(
+        customRouteRootIndex && customRouteRootIndex._value,
+        "should create a proper json for custom route with an extension"
+      )
+
       t.end()
     }
   )
@@ -140,7 +151,7 @@ test.cb("phenomic loader can be used with plugins", (t) => {
           "dumb"
         )
       })
-      t.plan(2+4) // 2, err, warn, 4 => array
+      t.plan(2+5) // 2, err, warn, 5   => array
       t.end()
     }
   )

src/phenomic-loader/index.js

@@ -52,13 +52,15 @@ module.exports = function(input: string) {
     })
   }, {})
 
-  const tmpUrl = urlify(
+  let tmpUrl = urlify(
     pluginsResult.head && pluginsResult.head.route
       // custom route
       ? pluginsResult.head.route
       // default route
       : relativePath
   )
+  tmpUrl = (tmpUrl.substring(0, 1) === "/") ? tmpUrl.slice(1) : tmpUrl
+
   const url = urlify(tmpUrl)
   const resourceUrl = urlify(tmpUrl, true)
   const contentHash = loaderUtils.getHashDigest(input)