Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

added the notion of router hierarchy was #82 #84

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

MarkNijhof
Copy link

Now all routers are automatically added to a list and enabled the use of this.navigate('../other_route') which then takes the parent route to execute navigate on. Not tested but this should also enable this.navigate('../../other/route'). Works as well with [Link href=../other_route]home[/Link]

Now all routers are automatically added to a list and enabled the use of this.navigate('../other_route') which then takes the parent route to execute navigate on. Not tested but this should also enable this.navigate('../../other/route'). Works as well with [Link href=../other_route]home[/Link]
@STRML
Copy link
Owner

STRML commented Sep 16, 2014

Thanks. I need some time to verify this and the tests but it looks pretty good to me.

@MarkNijhof
Copy link
Author

No problem, the code has changed a bit since last commit you (maybe) saw, now it jumps straight to the correct syntax instead of walking the tree back up. The tests now have 3 levels of routers and I walk from the nested ones back up one level and the double nested one I walk back up 2 levels

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants