feat(redirects): Support relative redirects for deeply nested config

docs/overview/redirect.md

@@ -26,3 +26,58 @@ const routes: Routes = [
 ```
 
 Now when a user navigates to `/blog/:id` they are automatically redirected to `/post/:id`. No more broken links!
+
+
+### Index Redirects
+If you want to redirect only if the path matches a specific route, you can use an index redirect:
+
+```ts
+const routes: Routes = [
+  {
+    path: '/users',
+    component: UsersComponent,
+    index: {
+      redirectTo: '/'
+    },
+    children: [
+      {
+        path: ':id',
+        component: UserByIdComponent
+      }
+    ]
+  }
+];
+```
+
+Now if a user goes to `/users` they will be redirected home but they can still navigate to `/users/:id` safely.
+
+
+### Relative Redirects
+You can also redirect relatively. This is handy for deeply nested links:
+
+```ts
+const routes: Routes = [
+  {
+    path: '/blog',
+    component: BlogComponent,
+    children: [
+      {
+        path: ':id',
+        component: PostComponent,
+        children: [
+          {
+            path: 'change',
+            redirectTo: 'edit'
+          },
+          {
+            path: 'edit',
+            component: EditPostComponent
+          }
+        ]
+      }
+    ]
+  }
+];
+```
+
+If a user navigates to `/blog/123/change` they will be redirected to `/blog/123/edit`.

lib/redirect.ts

@@ -58,12 +58,9 @@ export class RedirectHook implements Hook<Match> {
       const pattern = route.path || '';
 
       parentPattern = pattern.replace(/\/*$/, '/') + parentPattern;
-
-      if ( pattern.indexOf('/') === 0 )
-        break;
     }
 
-    return '/' + parentPattern;
+    return parentPattern;
   }
 }
 

spec/redirect.spec.ts

@@ -108,4 +108,28 @@ describe('Redirect Middleware', function() {
     expect(observer.next).not.toHaveBeenCalled();
     expect(router.replace).toHaveBeenCalledWith('/posts/543', {});
   });
+
+  it('should relatively redirect if the redirect path is relative', function() {
+    redirect.apply(routeSet$).subscribe(observer);
+
+    routeSet$.next({
+      routes: [
+        { path: '/blog' },
+        { path: ':id' },
+        {
+          path: 'edit',
+          redirectTo: 'change'
+        }
+      ],
+      routeParams: { id: '543' },
+      queryParams: {},
+      locationChange: {
+        type: 'push',
+        path: ''
+      }
+    });
+
+    expect(observer.next).not.toHaveBeenCalled();
+    expect(router.replace).toHaveBeenCalledWith('/blog/543/change', {});
+  });
 });