@tinkoff/router 0.1.67 → 0.1.70

package/README.md

@@ -60,8 +60,12 @@ export const myGuard: NavigationGuard = async ({ to }) => {
     return false; // block this transition
   }
 
-  if (to.config.redirect) {
-    return '/login/'; // call a redirect to the specified page
+  if (typeof to.config.redirect === 'string') {
+    return to.config.redirect; // call a redirect to the specified page
+  }
+
+  if (typeof to.config.needAuth && !isAuth()) {
+    return { url: '/login/', code: '302' }; // redirect to page with NavigateOptions
   }
 
   // if nothing is returned, the transition will be performed as usual
@@ -83,7 +87,7 @@ The behaviour of routing depends on the result of executing guards functions and
 - if all of the guards returns `undefined` than navigation will continue executing
 - if any of the guards returns `false` than navigation is getting blocked and next action differs on server and client
 - if any of the guards returns `string` it is considered as url to which redirect should be happen
-- if any of the guards returns `NavigateOptions` interface, `url` property from it is considered as url to which redirect should be happen
+- if any of the guards returns [`NavigateOptions`](#navigateoptions) interface, `url` property from it is considered as url to which redirect should be happen
 
 ### Transitions hooks
 
@@ -172,6 +176,13 @@ router.updateCurrentRoute({ query: { a: '1' } });
 2. `change`
 3. `afterUpdateCurrent`
 
+### NavigateOptions
+
+Object that allows to specify transition options both to [navigate](#navigate) and [updateCurrentRoute](#updatecurrentroute) transitions
+
+- `code` - redirect code that is returned on server in case of redirects
+- `navigateState` - any additional data that is stored with route
+
 ### Working with query
 
 #### query option

package/lib/index.browser.js

@@ -220,17 +220,16 @@ class AbstractRouter {
         await this.runHooks('afterUpdateCurrent', navigation);
     }
     async navigate(navigateOptions) {
-        return this.internalNavigate(navigateOptions, {});
+        return this.internalNavigate(typeof navigateOptions === 'string' ? { url: navigateOptions } : navigateOptions, {});
     }
     async internalNavigate(navigateOptions, { history }) {
         var _a;
-        const options = typeof navigateOptions === 'string' ? { url: navigateOptions } : navigateOptions;
-        const { url, replace, params, navigateState, code } = options;
+        const { url, replace, params, navigateState, code } = navigateOptions;
         const prevNavigation = (_a = this.currentNavigation) !== null && _a !== void 0 ? _a : this.lastNavigation;
         if (!url && !prevNavigation) {
             throw new Error('Navigate url should be specified and cannot be omitted for first navigation');
         }
-        const resolvedUrl = this.resolveUrl(options);
+        const resolvedUrl = this.resolveUrl(navigateOptions);
         const { to: from, url: fromUrl } = prevNavigation !== null && prevNavigation !== void 0 ? prevNavigation : {};
         let navigation = {
             type: 'navigate',
@@ -954,6 +953,15 @@ class Router extends ClientRouter {
         if (this.currentNavigation) {
             return this.delayNavigation(navigation);
         }
+        // ignore previous navigations that were put in delayedNavigation as we have more fresh navigation to execute
+        if (this.delayedNavigation) {
+            logger.info({
+                event: 'delay-navigation-drop',
+                delayedNavigation: this.delayedNavigation,
+                navigation,
+            });
+            this.delayedNavigation = null;
+        }
         return this.flattenDelayedNavigation(navigation);
     }
     delayNavigation(navigation) {

package/lib/index.es.js

@@ -220,17 +220,16 @@ class AbstractRouter {
         await this.runHooks('afterUpdateCurrent', navigation);
     }
     async navigate(navigateOptions) {
-        return this.internalNavigate(navigateOptions, {});
+        return this.internalNavigate(typeof navigateOptions === 'string' ? { url: navigateOptions } : navigateOptions, {});
     }
     async internalNavigate(navigateOptions, { history }) {
         var _a;
-        const options = typeof navigateOptions === 'string' ? { url: navigateOptions } : navigateOptions;
-        const { url, replace, params, navigateState, code } = options;
+        const { url, replace, params, navigateState, code } = navigateOptions;
         const prevNavigation = (_a = this.currentNavigation) !== null && _a !== void 0 ? _a : this.lastNavigation;
         if (!url && !prevNavigation) {
             throw new Error('Navigate url should be specified and cannot be omitted for first navigation');
         }
-        const resolvedUrl = this.resolveUrl(options);
+        const resolvedUrl = this.resolveUrl(navigateOptions);
         const { to: from, url: fromUrl } = prevNavigation !== null && prevNavigation !== void 0 ? prevNavigation : {};
         let navigation = {
             type: 'navigate',
@@ -682,13 +681,16 @@ class Router extends AbstractRouter {
         });
         return this.lastNavigation;
     }
-    async run(navigation) {
-        if (this.redirectCode) {
-            return this.redirect(navigation, { url: navigation.url.path, code: this.redirectCode });
-        }
+    async internalNavigate(navigateOptions, internalOptions) {
         // any navigation after initial should be considered as redirects
         if (this.getCurrentRoute()) {
-            return this.redirect(navigation, { url: navigation.url.path, code: navigation.code });
+            return this.redirect(this.lastNavigation, navigateOptions);
+        }
+        return super.internalNavigate(navigateOptions, internalOptions);
+    }
+    async run(navigation) {
+        if (this.redirectCode) {
+            return this.redirect(navigation, { url: navigation.url.href, code: this.redirectCode });
         }
         await super.run(navigation);
     }

package/lib/index.js

@@ -236,17 +236,16 @@ class AbstractRouter {
         await this.runHooks('afterUpdateCurrent', navigation);
     }
     async navigate(navigateOptions) {
-        return this.internalNavigate(navigateOptions, {});
+        return this.internalNavigate(typeof navigateOptions === 'string' ? { url: navigateOptions } : navigateOptions, {});
     }
     async internalNavigate(navigateOptions, { history }) {
         var _a;
-        const options = typeof navigateOptions === 'string' ? { url: navigateOptions } : navigateOptions;
-        const { url, replace, params, navigateState, code } = options;
+        const { url, replace, params, navigateState, code } = navigateOptions;
         const prevNavigation = (_a = this.currentNavigation) !== null && _a !== void 0 ? _a : this.lastNavigation;
         if (!url && !prevNavigation) {
             throw new Error('Navigate url should be specified and cannot be omitted for first navigation');
         }
-        const resolvedUrl = this.resolveUrl(options);
+        const resolvedUrl = this.resolveUrl(navigateOptions);
         const { to: from, url: fromUrl } = prevNavigation !== null && prevNavigation !== void 0 ? prevNavigation : {};
         let navigation = {
             type: 'navigate',
@@ -698,13 +697,16 @@ class Router extends AbstractRouter {
         });
         return this.lastNavigation;
     }
-    async run(navigation) {
-        if (this.redirectCode) {
-            return this.redirect(navigation, { url: navigation.url.path, code: this.redirectCode });
-        }
+    async internalNavigate(navigateOptions, internalOptions) {
         // any navigation after initial should be considered as redirects
         if (this.getCurrentRoute()) {
-            return this.redirect(navigation, { url: navigation.url.path, code: navigation.code });
+            return this.redirect(this.lastNavigation, navigateOptions);
+        }
+        return super.internalNavigate(navigateOptions, internalOptions);
+    }
+    async run(navigation) {
+        if (this.redirectCode) {
+            return this.redirect(navigation, { url: navigation.url.href, code: this.redirectCode });
         }
         await super.run(navigation);
     }

package/lib/router/abstract.d.ts

@@ -45,7 +45,7 @@ export declare abstract class AbstractRouter {
     protected internalUpdateCurrentRoute(updateRouteOptions: UpdateCurrentRouteOptions, { history }: InternalOptions): Promise<void>;
     protected runUpdateCurrentRoute(navigation: Navigation): Promise<void>;
     navigate(navigateOptions: NavigateOptions | string): Promise<void>;
-    protected internalNavigate(navigateOptions: NavigateOptions | string, { history }: InternalOptions): Promise<void>;
+    protected internalNavigate(navigateOptions: NavigateOptions, { history }: InternalOptions): Promise<void>;
     protected runNavigate(navigation: Navigation): Promise<void>;
     protected run(navigation: Navigation): Promise<void>;
     resolve(resolveOptions: NavigateOptions | string, options?: Parameters<AbstractRouter['resolveRoute']>[1]): import("../types").NavigationRoute;

package/lib/router/server.d.ts

@@ -8,6 +8,7 @@ export declare class Router extends AbstractRouter {
     constructor(options: Options);
     protected onRedirect: (navigation: Navigation) => Promise<void>;
     dehydrate(): Promise<Navigation>;
+    protected internalNavigate(navigateOptions: NavigateOptions, internalOptions: any): Promise<void>;
     protected run(navigation: Navigation): Promise<void>;
     protected redirect(navigation: Navigation, target: NavigateOptions): Promise<void>;
     protected notfound(navigation: Navigation): Promise<void>;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tinkoff/router",
-  "version": "0.1.67",
+  "version": "0.1.70",
   "description": "router",
   "main": "lib/index.js",
   "typings": "lib/index.d.ts",