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

Not working on route change #35

Closed
kgrvr opened this issue Jul 28, 2017 · 3 comments
Closed

Not working on route change #35

kgrvr opened this issue Jul 28, 2017 · 3 comments
Labels
needs repro Indicates that this issue needs a reproduction before further steps can be taken support Indicates a request for help / clarification

Comments

@kgrvr
Copy link

kgrvr commented Jul 28, 2017

I am changing route immediately after previewing notification box.

this._notify.notify('success', 'Customer updated!'); this._router.navigate(['/customer']);

In this situation above, notification is not working.
Plus after this, any other notification on other pages are also not previewing.

@dominique-mueller
Copy link
Owner

Can you make sure you've put the notification-container outside the pages / outside the router-outlet, for instance in your app component?

Would also be helpful if you could re-create this issue with a code example, perhaps with a Plunkr :)

@dominique-mueller dominique-mueller added needs repro Indicates that this issue needs a reproduction before further steps can be taken type: bug Indicates an unintended behaviour, unexpected problem, crash labels Jul 29, 2017
@kgrvr
Copy link
Author

kgrvr commented Aug 4, 2017

Ya exactly that was issue..needed to put tag in on the root component
Thanks for the reply :)

@kgrvr kgrvr closed this as completed Aug 4, 2017
@dominique-mueller
Copy link
Owner

Great that I was able to help!

@dominique-mueller dominique-mueller added support Indicates a request for help / clarification and removed type: bug Indicates an unintended behaviour, unexpected problem, crash labels Aug 4, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs repro Indicates that this issue needs a reproduction before further steps can be taken support Indicates a request for help / clarification
Projects
None yet
Development

No branches or pull requests

2 participants