fix: nester path generation & docs

README.md

@@ -166,13 +166,13 @@ To setup the plugin properly we recommend to put following snippet as part of `c
 
 ### Render
 
-`GET <host>/navigation/<id>?type=<type>`
+`GET <host>/navigation/render/<idOrSlug>?type=<type>`
 
 Return a rendered navigation structure depends on passed type (`tree`, `rfr` or nothing to render as `flat/raw`).
 
 *Note: The ID of navigation by default is `1`, that's for future extensions and multi-navigation feature.*
 
-**Example URL**: `https://localhost:1337/navigation/1`
+**Example URL**: `https://localhost:1337/navigation/render/1`
 
 **Example response body**
 
@@ -201,7 +201,7 @@ Return a rendered navigation structure depends on passed type (`tree`, `rfr` or
 ]
 ```
 
-**Example URL**: `https://localhost:1337/navigation/1?type=tree`
+**Example URL**: `https://localhost:1337/navigation/render/1?type=tree`
 
 **Example response body**
 
@@ -237,7 +237,7 @@ Return a rendered navigation structure depends on passed type (`tree`, `rfr` or
 ]
 ```
 
-**Example URL**: `https://localhost:1337/navigation/1?type=rfr`
+**Example URL**: `https://localhost:1337/navigation/render/1?type=rfr`
 
 **Example response body**
 

package.json

@@ -1,6 +1,6 @@
 {
   "name": "strapi-plugin-navigation",
-  "version": "1.0.0-beta.2",
+  "version": "1.0.0-beta.3",
   "description": "Strapi - Navigation plugin",
   "strapi": {
     "name": "Navigation",

services/__tests__/navigation.test.js

@@ -0,0 +1,4 @@
+
+describe('Dummy test', () => {
+  test('Dummy', () => { });
+});

services/navigation.js

@@ -211,16 +211,17 @@ module.exports = {
       switch (type) {
         case renderType.TREE:
         case renderType.RFR:
-          const itemParser = (item, path, field) => {
+          const itemParser = (item, path = '', field) => {
             const isExternal = item.type === itemType.EXTERNAL;
-            const parentPath = isExternal ? undefined : `${path}/${item.path}`;
+            const parentPath = isExternal ? undefined : `${path === '/' ? '' : path}/${item.path === '/' ? '' : item.path}`;
+            const slug = isString(parentPath) ? slugify((first(parentPath) === '/' ? parentPath.substring(1) : parentPath).replace(/\//g, '-')) : undefined;
             return {
               title: item.title,
               menuAttached: item.menuAttached,
               path: isExternal ? item.externalPath : parentPath,
               type: item.type,
               uiRouterKey: item.uiRouterKey,
-              slug: isString(parentPath) ? slugify(parentPath.replace('/', '-')) : undefined,
+              slug: !slug && item.uiRouterKey ? slugify(item.uiRouterKey) : slug, 
               external: isExternal,
               related: isExternal ? undefined : first(item.related),
               audience: !isEmpty(item.audience) ? item.audience.map(aItem => aItem.key) : undefined,