Default route not working

[YerkoPalma]

Expected behavior

Defining a route for '/404' should set a default route (not found)

Actual behavior

Default route did not work for me.

Steps to reproduce behavior

Set these routes

app.route('/', mainView)
app.route('/404', notFoundView)
app.mount('body')

Then visit any route different for '/', it will render mainView anyway.

I think it might be related with wayfarer#44.

[yoshuawuyts]

Oh yeah sounds like it. Oops haha

…

On Tue, Apr 4, 2017 at 8:01 PM Yerko Palma ***@***.***> wrote: Expected behavior Defining a route for '/404' should set a default route (not found) Actual behavior Default route did not work for me. Steps to reproduce behavior Set these routes app.route('/', mainView)app.route('/404', notFoundView)app.mount('body') Then visit any route different for '/', it will render mainView anyway. I think it might be related with wayfarer#44 <choojs/wayfarer#44>. — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#466>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ACWlesXR3OmFeLUTLXY7q7Meku4HJ-_dks5rsoVtgaJpZM4MzN9M> .

[expalmer]

What about app.route('/*', notFoundView)... It works for me

[YerkoPalma]

@expalmer just tried that, and it works as a default view. IMO this is still a workaround more than a solution, but is good to know.

[YerkoPalma]

@yoshuawuyts looks like in choo@6 the wildcard route will be the only default, so I think this issue can be closed now 👍

[yoshuawuyts]

woot!

…

On Thu, Jun 29, 2017 at 11:40 PM Yerko Palma ***@***.***> wrote: @yoshuawuyts <https://github.com/yoshuawuyts> looks like in ***@***.*** the wildcard route will be the only default, so I think this issue can be closed now 👍 — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#466 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ACWleitlogZ2elNd991FqJV8fS_39LXbks5sJBnBgaJpZM4MzN9M> .

[YerkoPalma]

I'm saying that this PR #521 resolves this issue. Am I wrong?

[yoshuawuyts]

I think this should be resolved, yeah - but not entirely positive because we need a test to see if '/*' and '/' resolve differently on root. PR welcome over on bankai for the test https://github.com/yoshuawuyts/wayfarer/blob/master/test/router.js