"Route not found" event about router5

[ivan-kleshnin]

Ooops, I see it's even stranger.

Invoking the .start([startPathOrState ,] done) function will:
Navigate to the default route if the current URL does not match an existing route

First. It's not documented how startPathOrState variable should affect.
Ok, let's skip this for a moment...

I have this code:

import {Router5} from "router5";

let router = new Router5(undefined, {
  useHash: false,      
  defaultRoute: "ginger",
});

router
  .addNode("home", "/")
  .addNode("ginger", "/ginger");

function handleAll(toState, fromState) {
  console.log("handleAll()");
  console.log("toState:", toState);
  console.log("fromState:", fromState);
}

router.addListener(handleAll);
router.start();

Let's try what this code does at access to http://localhost:3000/sdfdsf...

It suddenly redirects me to http://localhost:3000/sdfdsf/ginger 😧

Expected (at least to me) behavior would be to raise "Not found" event.
But... even following the docs... why not "http://localhost:3000/ginger"?

Is this a bug or... magic?

[troch]

For not found: http://router5.github.io/docs/transition.html

Error: .onTransitionError(cb), .offTransitionError(cb)
Listeners are of the following form: function (toState, fromState) { }. Transition error listeners are called with an extra err argument.

[troch]

http://localhost:3000/sdfdsf => http://localhost:3000/sdfdsf/ginger

Well it is not really a bug, but it could definitely be improved. So far the router assumes that if you managed to load http://localhost:3000/sdfdsf, then /sdfdsf might be your base path.

[ivan-kleshnin]

@troch if I just access /sdfdsfsdf onTransitionError is not invoked because redirect-to-default-route behavior overrides it.

Next I try:

function handleErrors(err) {
  console.log("error:", err);
}
router.onTransitionError(handleErrors);
router.navigate("dfdf");

handleErrors is still not called. Nothing happens.

What am I doing wrong?

[troch]

function handleErrors(toState, fromState, err) {
  console.log("error:", err);
}

[troch]

I know why it is not called, needs to be fixed

[troch]

Thanks for raising those issues 👍 , a few points have been improved with 0.6.0 (just released).

[troch]

Oops didn't mean to reopen

[ivan-kleshnin]

@troch, thank you for this project! I really appreciate your work.

[troch]

Thanks for your kind words!