Skip to content

Upgraded to alpha 29 #23

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

Closed
wants to merge 2 commits into from
Closed

Upgraded to alpha 29 #23

wants to merge 2 commits into from

Conversation

vcardins
Copy link

@vcardins vcardins commented Jul 7, 2015

Hi, still having issue with LoggedInRouterOutlet. I had to extend from routerMod.RouterOutlet once directly to RouterOutlet didn't work same for the activate method.
Another strange issue is the view, it is showing duplicated (http://screencast.com/t/j7XLRbzeXU).
I'll keep looking on that, sent the PR quickly in case you can figure it out sooner.
Thanks !

@mgonto
Copy link
Contributor

mgonto commented Jul 7, 2015

It's repated beacuse you're importing both the LoggedInRouterOutlet as well as the RouterOutlet in the app.ts. Remove the RouterOutlet and that should work.

Regarding the extend no idea why it doesn't work. hmm.

@vcardins
Copy link
Author

vcardins commented Jul 7, 2015

Actually it was not exactly that, just updated the PR. No luck yet with the LoggedInRouterOutlet activate method.

if (uri === '' || uri === '/') {
      router.navigate('/home');
 } else {
      //router.navigate(uri); This line is causing the repetition. Any thoughts?
 }

@PatrickJS
Copy link
Contributor

I have a working version here #24

@PatrickJS
Copy link
Contributor

this can be closed

@mgonto mgonto closed this Jul 11, 2015
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.

3 participants